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

Fix spelling errors #22

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

dforsi
Copy link

@dforsi dforsi commented Jul 9, 2022

This PR is the result of automatic edits made with the tool codespell.
It fixes one small spelling error in manpages/man1/jtdx.1.txt and changes other two files, but It breaks the formatting in package_description.txt I'm not sure ho to handle this, all lines should be reformatted?

The codespell shows more words that are possible errors, but it needs manual work to exclude some directories and some false positives; a starting point is:
codespell --summary --skip=*.csv,*.dat,translations --ignore-words-list=ba,fo,inout,nd,rcall

I can modify this PR or create a new PR but I'm not sure what directories should be skipped because contain files form upstream projects.

Fixed with:
codespell --write-changes CMakeLists.txt  manpages/ package_description.txt
@brianjester
Copy link

If this PR is closed (not accepted) is the only result that there are some spelling errors, and nothing is broke?

@dforsi
Copy link
Author

dforsi commented Nov 13, 2022 via email

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