Skip to content

Update pre-commit - #851

Closed
forsyth2 wants to merge 1 commit into
mainfrom
update-pre-commit
Closed

Update pre-commit#851
forsyth2 wants to merge 1 commit into
mainfrom
update-pre-commit

Conversation

@forsyth2

Copy link
Copy Markdown
Collaborator

Summary

Objectives:

  • Update pre-commit dependencies based on running pre-commit autoupdate

Issue resolution:

Select one: This pull request is...

  • a bug fix: increment the patch version
  • a small improvement: increment the minor version
  • a new feature: increment the minor version
  • an incompatible (non-backwards compatible) API change: increment the major version

Small Change

  • To merge, I will use "Squash and merge". That is, this change should be a single commit.
  • Logic: I have visually inspected the entire pull request myself.
  • Pre-commit checks: All the pre-commits checks have passed.

@forsyth2 forsyth2 self-assigned this Aug 13, 2026
@forsyth2 forsyth2 added the DevOps CI/CD, configuration, etc. label Aug 13, 2026
@forsyth2
forsyth2 requested a lite review from Copilot August 13, 2026 18:11

Copilot AI 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.

Pull request overview

Updates the repository’s pre-commit tooling versions to match the output of
pre-commit autoupdate, aligning local QA tools with hook revisions used during
development.

Changes:

  • Bumped black and mypy versions in the conda dev environment.
  • Updated .pre-commit-config.yaml hook revisions (notably pre-commit-hooks,
    black, isort, mypy).

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
conda/dev.yml Updates QA tool pins (black, mypy) used in the dev conda environment.
.pre-commit-config.yaml Updates hook rev pins used by pre-commit (including isort to a beta rev).

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread .pre-commit-config.yaml
Comment on lines 19 to 22
- repo: https://github.com/PyCQA/isort
rev: 6.0.1
rev: 9.0.0b2
hooks:
- id: isort
Comment thread conda/dev.yml
Comment on lines +41 to +44
- black ==26.5.1
- flake8 ==7.3.0
- isort ==6.0.1
- mypy ==1.18.2
- mypy ==2.3.0
@forsyth2 forsyth2 added the semver: bug Bug fix (will increment patch version) label Aug 13, 2026
Comment thread .pre-commit-config.yaml
# Can run individually with `pre-commit run isort --all-files`
- repo: https://github.com/PyCQA/isort
rev: 6.0.1
rev: 9.0.0b2

@xylar xylar Aug 13, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Suggested change
rev: 9.0.0b2
rev: 8.0.1

See 8.0.1

Comment thread conda/dev.yml
- black ==25.1.0
- black ==26.5.1
- flake8 ==7.3.0
- isort ==6.0.1

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Suggested change
- isort ==6.0.1
- isort ==8.0.1

@forsyth2

Copy link
Copy Markdown
Collaborator Author

Replaced by #854, closing this PR.

@forsyth2 forsyth2 closed this Aug 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

DevOps CI/CD, configuration, etc. semver: bug Bug fix (will increment patch version)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants