lint: support the existence of 'venv'
Metadata
Current evaluation
Merged into main. Added 'venv' to .gitignore and codespell skip list to prevent linting errors for common virtual environment directories. Approved by one reviewer with passing CI checks.
Suggested action: —
No scores available.
Issue body
I think 'venv' is a very common virtual-env name, so add it to gitignore and have codespell skip it. I'm actually not sure if this type of change is desirable, so feel free to reject it!
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged into main. Added 'venv' to .gitignore and codespell skip list to prevent linting errors for common virtual environment directories. Approved by one reviewer with passing CI checks. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Proposed adding 'venv' to .gitignore and codespell exclusions. Closed without merging, indicating rejection or abandonment. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged into main. Adds 'venv' to .gitignore and configures codespell to skip it, resolving linting warnings for common virtual environment directories. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #3941 lint: support the existence of 'venv' | snapcraft | merged | Merged after reviewer approval and passing CI. Adds 'venv' and '.venv' to .gitignore and configures codespell to skip these common virtual environment directories. | |
| #4022 lint: ignore snapd-testing-tools, venv | snapcraft | merged | Merged linter configuration updates to ignore venv, snapd-testing-tools, and cache directories, resolving local development failures. Approved by reviewers, passed CI, and integrated into main. | |
| #461 Add .venv removal to make clean | starbase | merged | Merged. Added .venv to the make clean target in common.mk to enable full virtual environment recreation. Approved by three reviewers, passed CI checks, and resolves canonical/starbase#460. |