Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

everywhere: cleanup unused/unnecessary spellcheck-disabling #6295

Merged
merged 14 commits into from
Apr 30, 2024

Conversation

BenWiederhake
Copy link
Collaborator

We have lots of unnecessary spell-checker:disable-THING all over the place, 44 in fact. That means we wouldn't notice typos in those lines, so let's remove unnecessary stuff like that.

I found this when watching @jadijadi's video about #6225, because he copy-pastes a non-sensical spell-checker:disable-line on an empty line.

Code to find the unnecessary spellcheck-statements: https://gist.github.com/BenWiederhake/f43c386c957825b6109325c39a351cac

In the process, I also found that tests/by-util/test_cat.rs could be simplified a little bit. Not sure if it's worth it, but now it's done.

@sylvestre sylvestre merged commit 58d98df into uutils:main Apr 30, 2024
63 of 65 checks passed
Copy link

GNU testsuite comparison:

Skipping an intermittent issue tests/tail/inotify-dir-recreate (passes in this run but fails in the 'main' branch)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants