Skip to content

Conversation

@harryswift01
Copy link
Member

Summary

This PR improves CI reliability and clarity by fully isolating the test, documentation, and pre-commit environments. Each CI job now installs only the dependencies required for its specific purpose, removing unnecessary coupling and preventing unrelated dependency failures from surfacing in the wrong job.

Changes

Isolate CI jobs and dependency scopes:

  • Updated the CI workflow so that test, documentation, and pre-commit jobs install only their respective dependency extras.
  • Removed implicit dependency coupling between CI jobs.
  • Ensured failures are confined to the job responsible for the failing component.

Align packaging extras with CI and pre-commit usage:

  • Refined optional dependency groups in pyproject.toml to clearly separate testing, docs, and pre-commit concerns.
  • Added explicit linting and formatting tools to the pre-commit extra to match the configured pre-commit hooks.
  • Made development tooling dependencies explicit and easier to reason about.

Impact

  • CI failures are easier to diagnose and attribute to the correct source.
  • Documentation or testing dependency changes no longer affect pre-commit checks.
  • Contributors can install only the tooling they need for a given task.
  • Improves long-term maintainability through clearer separation of concerns.

@harryswift01 harryswift01 self-assigned this Jan 21, 2026
@harryswift01 harryswift01 added the bug Something isn't working label Jan 21, 2026
@harryswift01 harryswift01 linked an issue Jan 21, 2026 that may be closed by this pull request
@harryswift01 harryswift01 changed the title Isolate CI Testing Strategies For pre-commit, docs and testing Isolate CI Testing Strategies for pre-commit, docs and testing Jan 21, 2026
@harryswift01 harryswift01 requested a review from jimboid January 21, 2026 09:43
Copy link
Member

@jimboid jimboid left a comment

Choose a reason for hiding this comment

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

Great spot Harry! Can you add the other two maintained packages to your list and check it is not the same there.

@harryswift01 harryswift01 merged commit bef4f99 into main Jan 21, 2026
14 checks passed
@harryswift01 harryswift01 deleted the 256-bug-project-ci branch January 21, 2026 10:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Project CI does not isolate pre-commit and docs testing

3 participants