This repository has been archived by the owner on Dec 16, 2022. It is now read-only.
flake8 and darglint-ignore-regex #180
Labels
Comments
s-weigand
added a commit
to s-weigand/pyglotaran
that referenced
this issue
Oct 22, 2021
jsnel
pushed a commit
to glotaran/pyglotaran
that referenced
this issue
Oct 22, 2021
* 🔧👌 Move up yesqa hook, to remove noqa's before linting * 🔧♻️ Moved pydocstyle config to tox.ini * 🔧 ♻️ Moved up flake8 hook directly after yesqa hook * 🔧⬆️ Updated pre-commit config * 🔧♻️ Run Doc linting tools via flake8 This allows a uniform usage of 'noqa's, which is also compatible with yesqa * 🔧🩹 Remove parameter.py from linter exceptions * 🔧 Fixed darglint version to 1.8.0 due to bug in 1.8.1 Ref: terrencepreilly/darglint#180
|
Yes, it doesn't parse this value from config files. You're welcome to submit a pull request if you think it should. |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Looks like flake8 has added the cmd line option of --darglint-ignore-regex.
When putting that in the tox.ini file such as
[darglint]
darglint-ignore-regex = ^_(.*)
darlint isn't picking up the regex.
If put the
--darglint-ignore-regex='^_(.*)'darglint is picking that up.The text was updated successfully, but these errors were encountered: