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

ci: suppress codespell warnings about fixes disabled in dictionary #775

Merged
merged 3 commits into from
Feb 8, 2020

Conversation

AirQuick
Copy link
Member

@AirQuick AirQuick commented Feb 7, 2020

The latest codespell warns a lot about thead ==> thread disabled in dictionary:

$ pip install --upgrade git+https://github.com/codespell-project/codespell.git
...
Successfully installed codespell-1.17.0.dev0
$ codespell --quiet-level 2 --skip=".git,./src/schematron/iso-schematron"
./src/reporter/format-xspec-report.xsl:223: thead  ==> thread  | disabled due to the html tag
./src/reporter/format-xspec-report.xsl:232: thead  ==> thread  | disabled due to the html tag
./src/reporter/format-xspec-report.xsl:328: thead  ==> thread  | disabled due to the html tag
./src/reporter/format-xspec-report.xsl:338: thead  ==> thread  | disabled due to the html tag
./test/xspec-param-position.xspec:20: poisition  ==> position
...

The warnings are too many. To suppress it, this pull request adds bitmask 4 to QUIET_LEVEL:

$ codespell --quiet-level 6 --skip=".git,./src/schematron/iso-schematron"
./test/xspec-param-position.xspec:20: poisition  ==> position

(poisition is addressed in #774.)

@AirQuick AirQuick added the test label Feb 7, 2020
@AirQuick AirQuick added this to the v1.6.0 milestone Feb 7, 2020
@commit-lint
Copy link

commit-lint bot commented Feb 7, 2020

Continuous Integration

  • suppress codespell warnings about fixes disabled in dictionary (355d791)

Contributors

@AirQuick

@AirQuick
Copy link
Member Author

AirQuick commented Feb 8, 2020

Merging this to facilitate local pre-commit hook.

@AirQuick AirQuick merged commit b32f71f into xspec:master Feb 8, 2020
@AirQuick AirQuick deleted the codespell-suppress-dicdiswarn branch February 8, 2020 15:08
magwas pushed a commit to kode-konveyor/xspec that referenced this pull request Mar 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant