Audit flake8 CI errors vs. warnings
Nobody has claimed this yet.
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 48/100
Research direction
Start by reading workflows/lint.yml and the existing pre-commit configuration. Compare how flake8 handles F401 and F841 with the checks already prohibited by pre-commit, then determine whether the CI configuration needs adjustment. Done means the two checks have a clear, documented CI decision and the relevant lint configuration is consistent.
Written by the indexing model from the issue text.
Description
#1 mentions that the flake8 in workflows/lint.yml may be overly permissive. This could result in bad code getting in.
- evaluate error on F401 for unused imports
- evaluate error on F841 for local variable assigned but unused
Note that pre-commit config already prohibits both of these.
- Dominant language
- Python
- Stars
- 0
- Forks
- 0
- PR merge metrics
- No merged PRs in 30d
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Similar issues
-
documentation help wanted
Difficulty 2/5 1-3 hours Newbie friendliness 90/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 90/100
simonw/sqlite-utils#872 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 88/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 78/100