Skip to content

ci(P-4): bump pre-commit isort pin 5.13.2 → 8.0.1#42

Merged
pcalnon merged 1 commit intomainfrom
fix/p4-bump-isort-8-0-1
May 3, 2026
Merged

ci(P-4): bump pre-commit isort pin 5.13.2 → 8.0.1#42
pcalnon merged 1 commit intomainfrom
fix/p4-bump-isort-8-0-1

Conversation

@pcalnon
Copy link
Copy Markdown
Owner

@pcalnon pcalnon commented May 3, 2026

Aligns this repo's pre-commit isort pin with the local-Juniper-conda baseline (8.0.1) and juniper-canopy (8.0.1).

Closes the local-vs-pre-commit drift that caused two unnecessary CI iterations during V38 (juniper-ml notes/POST_V38_OPEN_ISSUES_PLAN_2026-05-03.md P-4).

pre-commit run isort --all-files clean — pure pin bump, no reformat.

Aligns the pre-commit pin with what local Juniper conda envs ship
(isort 8.0.1) and with juniper-canopy's pre-commit pin (already
8.0.1 since 2026-04-06's Phase 5 housekeeping).

Previously, devs running ``isort`` directly from a Juniper conda
env produced output that disagreed with what CI's pre-commit
applied — see notes/POST_V38_OPEN_ISSUES_PLAN_2026-05-03.md (P-4),
which traced two unnecessary CI iterations to this drift during
the V38 diagnostic cycle.

``pre-commit run isort --all-files`` is clean against the new pin
on this repo's tracked files, so this is a pure pin bump with no
functional reformat.
@pcalnon pcalnon self-assigned this May 3, 2026
Copy link
Copy Markdown
Owner Author

@pcalnon pcalnon left a comment

Choose a reason for hiding this comment

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

approved

@pcalnon pcalnon merged commit c7e6e54 into main May 3, 2026
38 checks passed
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