Skip to content

Add a test that --write-changes skips disabled automatic fixes - #4058

Open
ekanshul wants to merge 1 commit into
codespell-project:mainfrom
ekanshul:test-disabled-autofix-1791
Open

ekanshul wants to merge 1 commit into
codespell-project:mainfrom
ekanshul:test-disabled-autofix-1791

Conversation

@ekanshul

Copy link
Copy Markdown
Contributor

Fixes #1791.

Dictionary lines with a reason after the last comma (no trailing comma) are still supported for custom dictionaries. --write-changes must report those matches and leave the file unchanged.

The new test uses a custom dictionary with the README example (clas->class, disabled because of name clash in c++) and checks both the dry-run report and -w.

Dictionary entries with a reason after the last comma must be reported
but not rewritten. This is still supported for custom dictionaries.
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.

Add a test for disable automatic fix in the dictionary file

1 participant