Skip to content

chore(style): Add style linting reported by Flake8#176

Open
olegsta wants to merge 7 commits into
vsys-host:mainfrom
olegsta:add_stylelint
Open

chore(style): Add style linting reported by Flake8#176
olegsta wants to merge 7 commits into
vsys-host:mainfrom
olegsta:add_stylelint

Conversation

@olegsta

@olegsta olegsta commented Jan 15, 2026

Copy link
Copy Markdown
Contributor
  • Add style linting support using flake8
  • Integrate lint checks into CI workflow
  • Cleane up import statements (replaced wildcard imports with explicit ones)
  • Applie consistent code formatting across the project
  • Remove unused imports and simplified expressions
  • Improve code readability and lint compliance

- Add flake8 for static Python code analysis and PEP8 compliance
- Introduce .flake8 configuration to define project-wide linting rules
- Integrate flake8 check into CI pipeline to fail builds on linting errors
- Improve overall code quality, readability, and early detection of potential bugs
- Reformat code for readability
- Normalize quotes and line wrapping
- No schema or constraint logic changes
- Auto-fix f-string placeholders (F541) across the project
- Normalize formatting for readability
- No changes to business logic, database schema, or constraints
- Delete all unused imports detected by Flake8 (F401)
- Improve code readability by cleaning up unnecessary lines
- No changes to business logic, database schema, or constraints
- Normalization of import ordering and formatting
- Remove wildcard import from shkeeper.models
- Explicitly import only required model classes
- Improve code readability and import clarity
- Prevent potential name collisions and linting issues (F403/F405)
- No changes to business logic or runtime behavior
- remove unused exception variables in except blocks
- keep logging via logger.exception without binding exception
- Remove unused imports
- fix ci tests
- No changes to business logic or functionality
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