summaryrefslogtreecommitdiff
path: root/maint/add_warning
Commit message (Collapse)AuthorAgeFilesLines
* add_warnings: ignore clippy::result_large_errNick Mathewson2022-11-031-0/+1
| | | | | This is a temporary workaround for #587 until we get it fixed for real.
* add_warnings: work with more non-GNU finds.Nick Mathewson2022-11-031-1/+1
| | | | | We were doing `find -name '*.rs'`, but `find . -name '*.rs'` (with the .) is more standard.
* Allow "clippy::single_char_pattern" in tests.Ian Jackson2022-10-121-0/+1
| | | | | | | This lint exists for perf reasons, and this is rarely relevant in tests. Using double quoted str is generally cognitively less burdensome.
* format python script in maintFAMASoon2022-08-171-14/+25
|
* maint/add_warning: Fix parsing of --checkIan Jackson2022-06-241-1/+1
| | | | Previously this would eat the first filename!
* clippy: Consolidate many lints in maint/add_warningIan Jackson2022-06-241-0/+3
| | | | | | Found these by disabling the nightly dbg macro special case. Now, we have a mechanism for globally adding suppressions to tests, we can use that instead.
* maint/add_warning: Add two lints via add_warning as a test caseIan Jackson2022-06-241-0/+2
|
* maint/add_warning: Add ability maintain test lint blocksIan Jackson2022-06-241-8/+34
|
* maint/add_warning: Scan all files by default, but insist only in someIan Jackson2022-06-241-9/+16
| | | | | | | This will allow us to have add_warning manage test lint blocks. We have to stop printing all the filenames because there are too many. Filenames still come out on error of course.
* clippy: Supresss clippy::significant_drop_in_scrutineeIan Jackson2022-06-231-0/+1
| | | | As per the linked discussion.
* lints: Scheme for allowing us to name nightly lintsIan Jackson2022-06-231-1/+3
| | | | | As per https://gitlab.torproject.org/tpo/core/arti/-/issues/501
* maint/add_warning: Convert remaining assertsIan Jackson2022-05-311-4/+8
|
* maint/add_warning: Introduce ImproperFile and convert one assertIan Jackson2022-05-311-2/+18
|
* maint/add_warning: Use python3 argparseIan Jackson2022-05-311-11/+9
|
* maint/add_warning: Add "how to fix" to error messageIan Jackson2022-05-311-0/+1
| | | | | As per https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/557#note_2808879
* maint/add_warning: Provide --check modeIan Jackson2022-05-311-2/+22
|
* maint/add_warning: Change shebang to python3Ian Jackson2022-05-311-1/+1
| | | | "python" sometimes means Python 2.
* maint/add_warning: Use .tmp~ for temp filename, not .bakIan Jackson2022-05-311-3/+3
| | | | | "bak" ought to be a backup file, with the old contents, which this most definitely isn't.
* lints: Add let_unit_value allow to add_warningIan Jackson2022-05-311-0/+1
| | | | | add_warning can now add allows. This one was agreed in some other MR for one crate. Allow it everywhere.
* lints: Add two missing ones to add_warningIan Jackson2022-05-311-0/+2
| | | | | These were in some crates, but not all. Put them in add_warning where they will propagate everywhere.
* lints: maint/add_warning: New marker-based modeIan Jackson2022-05-311-35/+21
| | | | | | | | We don't process the in-script lint block, just paste it in, now. This is less complicated. This reverts/replaces "maint/add_warnings: Make it add the markers for its future self"
* maint/add_warnings: Make it add the markers for its future selfIan Jackson2022-05-311-0/+6
| | | | We'll run this in a moment. Then we'll revert it.
* lints: maint/add_warning: Improve orderingIan Jackson2022-05-311-1/+1
| | | | | The crates we have aren't consistent. But changing this means the deltas resulting from the script are smaller.
* maint/add_warnings: Tidy up, and add some commentaryIan Jackson2022-05-311-2/+7
| | | | | | This puts the actually-used lint list at the top. No functional change.
* add_warning: remove clone_on_ref_ptrNick Mathewson2022-03-011-1/+0
| | | | See !352
* maint/add_warning: Remove duplicate argv0 from usage messageIan Jackson2022-02-181-1/+1
|
* rename *.py scripts to bare nameIan Jackson2022-02-181-0/+118
Update all references. There are three remaining hits for git-grep '\.py\b' all of which are scripts in "little-t-tor", not here in arti.