Skip to content

chore: add more linters#20

Merged
danfimov merged 2 commits intomainfrom
add-more-linters
Oct 15, 2025
Merged

chore: add more linters#20
danfimov merged 2 commits intomainfrom
add-more-linters

Conversation

@danfimov
Copy link
Owner

No description provided.

Copilot AI review requested due to automatic review settings October 15, 2025 17:38
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds additional linting tools and security checks to improve code quality and development workflow. The changes reorganize dependency groups, update versions, and integrate multiple linting tools into the pre-commit hooks and CI/CD pipeline.

Key Changes

  • Reorganized dependency groups in pyproject.toml with separate lint, test, and docs groups
  • Added new linting tools: bandit (security), codespell (spelling), and zizmor (GitHub Actions security)
  • Enhanced pre-commit configuration with comprehensive hooks including security scanning and typo detection

Reviewed Changes

Copilot reviewed 9 out of 10 changed files in this pull request and generated no comments.

Show a summary per file
File Description
pyproject.toml Reorganized dependencies into logical groups and bumped version to 0.6.0
.pre-commit-config.yaml Added comprehensive pre-commit hooks for linting, security, and code quality
tests/conftest.py Added uvloop fixture for faster async tests
Makefile Enhanced lint targets with separate ruff and mypy commands
.github/workflows/*.yml Updated GitHub Actions to use pinned commit SHAs for security
docs/tutorial/common_issues.md Updated database connection string example
.github/dependabot.yml Added cooldown configuration for dependency updates

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@danfimov danfimov merged commit 59f46bf into main Oct 15, 2025
4 checks passed
@danfimov danfimov deleted the add-more-linters branch October 15, 2025 17:39
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.

2 participants