Skip to content

Conversation

@danfimov
Copy link
Contributor

Dependency and tooling updates:

  • Upgraded pre-commit version in pyproject.toml from ^2.20.0 to ^4.3.0, ensuring compatibility with newer hooks and features. Old version didn't work with typos hook for example.
  • Updated pre-commit hooks version in .pre-commit-config.yaml

New code quality and security checks:

  • Added typos and detect-secrets hooks to .pre-commit-config.yaml for typo detection and secret scanning, with custom exclusions for certain files and lines.
  • Configured typo detection exclusions in pyproject.toml to avoid false positives in documentation.

Minor documentation fix:

  • Fixed a typo in a comment in taskiq/cli/scheduler/run.py ("timzone" → "timezone").typo in code comment

@Copilot Copilot AI review requested due to automatic review settings October 23, 2025 09:47
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 enhances code quality by fixing a typo and implementing automated checks to prevent similar issues in the future. The changes upgrade the pre-commit tooling infrastructure and add new hooks for typo detection and secret scanning.

Key changes:

  • Fixed a typo in a code comment ("timzone" → "timezone")
  • Upgraded pre-commit from v2.20.0 to v4.3.0 to support modern hooks
  • Added typos and detect-secrets hooks with appropriate exclusion configurations

Reviewed Changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated no comments.

File Description
taskiq/cli/scheduler/run.py Corrected spelling of "timzone" to "timezone" in comment
pyproject.toml Upgraded pre-commit version and added typos configuration with documentation exclusions
.pre-commit-config.yaml Updated hook versions and added typos and detect-secrets hooks with custom exclusions

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

@s3rius s3rius merged commit b588dc6 into taskiq-python:master Oct 23, 2025
33 checks passed
@danfimov danfimov deleted the add-typos-hook branch October 23, 2025 10:31
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