-
Notifications
You must be signed in to change notification settings - Fork 476
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
support ignore for blocks of code #3381
Comments
I have recently created a PR that adds support for ignoring regions that span multiple lines: #3476. It works by allowing the user to specify a regex that is applied using So the begin/end tags are not hard coded, instead one sets them on the command line. For example with: one can ignore code blocks with:
or
|
@julian-smith-artifex-com Thanks that's really helpful. And I know it's easier said than done, but would it be possible for you (or anyone else) to allow the block ignore to take a pattern just like the inline ignore tag, like:
|
@julian-smith-artifex-com i am getting error on this, see below $ codespell --version
2.3.0
$ codespell --ignore-multiline-regex 'codespell:ignore-begin.*codespell:ignore-end'
codespell: error: unrecognized arguments: --ignore-multiline-regex trying to replace codespell/codespell_lib/_codespell.py Lines 437 to 449 in c6bdc1f
seems it is not included in version 2.3.0 :( |
The current release 2.4.0 on pypi.org seems to support See https://pypi.org/project/codespell/#history. It looks like it was released just a couple of days ago, on 2025-1.21. |
This is the follow-up issue of #1212. It is now possible to ignore a single line from a file by adding
# codspell: ignore
comment.Would it be possible to continue to mimick what black is doing and implement an on/off comment.
It's easier to understand with a real life example:
in my lib I need to gather some data from USDA wich associate a US state to a 2 letter code. I thus store a boring dictionnary at the start of my module:
When I run the code spell pre-commit, it falls on "ND" which is the code for Nevada and offers me to replace it with "AND" or "2ND".
The codespell ignore comments is perfectly doing it's job but I lack visibility. If you look carefully at my example, the comments
#fmt: off
and#fmt: on
are deactivating black for the whole length of the dict. Would it be possible to implement such a comment for codespell ?The text was updated successfully, but these errors were encountered: