Skip to content

Remove tests_check job from CI workflow#950

Merged
Carreau merged 1 commit into
mainfrom
claude/remove-tests-check-ci-8w9v2e
Jul 16, 2026
Merged

Remove tests_check job from CI workflow#950
Carreau merged 1 commit into
mainfrom
claude/remove-tests-check-ci-8w9v2e

Conversation

@Carreau

@Carreau Carreau commented Jul 16, 2026

Copy link
Copy Markdown
Member

Summary

Removed the tests_check job from the GitHub Actions workflow that was used for branch protection status checks.

Changes

  • Deleted the tests_check job that aggregated the status of all other test jobs using the alls-green action
  • This job was a no-op that only served to provide a single branch protection rule by collecting results from:
    • coverage
    • test_lint
    • test_docs
    • test_minimum_versions
    • test_prereleases
    • check_links
    • check_release
    • test_sdist

Notes

Branch protection rules should be updated to reference individual job statuses directly instead of relying on this aggregation job.

https://claude.ai/code/session_01CgaWwFHViCQUgmqTaCpjsq

The tests_check job was only used for branch protection and simply
aggregated the status of all other jobs. This redundant step can be
removed as branch protection rules can reference individual required jobs
directly.

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01CgaWwFHViCQUgmqTaCpjsq
@Carreau Carreau merged commit 5eeaae3 into main Jul 16, 2026
57 of 58 checks passed
@Carreau Carreau deleted the claude/remove-tests-check-ci-8w9v2e branch July 16, 2026 12:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants