Skip to content

chore: update flake8 configuration to exclude third-party directories#29

Merged
Kuonirad merged 2 commits intofeature/initial-setupfrom
devin/1734938357-update-flake8-config
Dec 23, 2024
Merged

chore: update flake8 configuration to exclude third-party directories#29
Kuonirad merged 2 commits intofeature/initial-setupfrom
devin/1734938357-update-flake8-config

Conversation

@devin-ai-integration
Copy link
Contributor

This PR updates the flake8 configuration to properly exclude third-party directories from linting:

  • Adds venv_311 to excluded directories
  • Adds node_modules to excluded directories
  • Maintains existing exclude patterns
  • Keeps current linting rules
  • Updates gitignore to prevent virtual environment tracking

This change ensures that flake8 only reports issues from our project's source files, not from third-party packages or virtual environments.

Link to Devin run: https://app.devin.ai/sessions/c6848175de354945a69903fcc437018b

- Update flake8 configuration to exclude third-party directories
- Add venv_311 and node_modules to excluded directories
- Add venv_311 to gitignore to prevent tracking virtual environment files
- Update CI workflow for flake8 configuration

Co-Authored-By: Kevin Kull <kevinkull.kk@gmail.com>
@devin-ai-integration
Copy link
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add "(aside)" to your comment to have me ignore it.
  • Look at CI failures and help fix them

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@Kuonirad Kuonirad merged commit 8bea8df into feature/initial-setup Dec 23, 2024
3 of 6 checks passed
@Kuonirad Kuonirad deleted the devin/1734938357-update-flake8-config branch December 23, 2024 07:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant