Skip to content

chore(ci): add flake8 lint workflow and fix minor lint issue #62

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Oct 5, 2025
Merged

chore(ci): add flake8 lint workflow and fix minor lint issue #62

merged 1 commit into from
Oct 5, 2025

Conversation

@DivInstance
Copy link
Contributor

@DivInstance DivInstance commented Oct 5, 2025

This PR adds flake8 as a CI lint job and includes a small code fix to ensure lint passes.

Changes:

  • Add .github/workflows/lint.yml to run flake8 on push and pull_request using Python 3.8, matching the Dockerfile base image.
  • Fix an unused exception variable in github-dork.py to satisfy flake8.

Notes:

  • The repository already defines flake8 settings in setup.cfg (max-line-length=120), which the workflow respects.
  • No functional behavior changed; only lint improvements and CI setup.

Resolves: #59

@DivInstance
chore(ci): add flake8 lint workflow and fix minor lint issue
21ed392 
- Add GitHub Actions workflow to run flake8 on push and PR
- Align Python version with Dockerfile (3.8)
- Fix unused variable in exception handler to satisfy flake8

Refs: techgaun#59
techgaun
techgaun approved these changes Oct 5, 2025
View reviewed changes
Copy link
Owner

@techgaun techgaun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👌 💜 🔥 💯 🚀 LGTM 🚀 💯 🔥 💜 👌

@DivInstance
Copy link
Contributor Author

DivInstance commented Oct 5, 2025

Thank you so much for merging :))
Have a great day ahead.

@techgaun techgaun merged commit 0cfd2cc into techgaun:master Oct 5, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@techgaun techgaun techgaun approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

U

2 participants

@DivInstance @techgaun