Skip to content

build: upgrade pre-commit to 4.3.0 and add dev dependencies#7733

Open
wtfiwtz wants to merge 3 commits into
getredash:masterfrom
orchestrated-io:build/pre-commit-4.3-dev-tools
Open

build: upgrade pre-commit to 4.3.0 and add dev dependencies#7733
wtfiwtz wants to merge 3 commits into
getredash:masterfrom
orchestrated-io:build/pre-commit-4.3-dev-tools

Conversation

@wtfiwtz

@wtfiwtz wtfiwtz commented Jun 2, 2026

Copy link
Copy Markdown

What type of PR is this?

  • Refactor

Description

Upgrade pre-commit and add transitive dev dependencies:

  • pre-commit: 3.3.3 → 4.3.0
  • Add filelock: >=3.20.3
  • Add pygments: >=2.20.0
  • Add virtualenv: >=20.36.1

These are development-time dependencies with no runtime impact. The new dependencies are transitive requirements of pre-commit 4.3.0.

How is this tested?

  • Pre-commit hooks - verified with pre-commit run --all-files
  • N/A (dev-only dependencies)

Related Tickets & Documents

Split from orchestrated-io/redash#7719 per @zachliu's review feedback to separate dev tooling upgrades from runtime dependency changes.

Part of the security vulnerability remediation work tracked in #7711.

Note

This PR is marked as draft until manual testing is complete.

Made with Cursor

Upgrade pre-commit and add transitive dev dependencies:

- pre-commit: 3.3.3 → 4.3.0
- Add filelock: >=3.20.3
- Add pygments: >=2.20.0
- Add virtualenv: >=20.36.1

These are development-time dependencies with no runtime impact. The new
dependencies are transitive requirements of pre-commit 4.3.0.

Related: split from orchestrated-io/redash#7719
Co-authored-by: Cursor <cursoragent@cursor.com>
@wtfiwtz wtfiwtz marked this pull request as ready for review June 3, 2026 21:33

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

No issues found across 4 files

Re-trigger cubic

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