Skip to content

Group fixture checks into single Code Analysis job - #8368

Merged
TomasVotruba merged 1 commit into
mainfrom
tv-group-fixture-checks
Aug 24, 2026
Merged

Group fixture checks into single Code Analysis job#8368
TomasVotruba merged 1 commit into
mainfrom
tv-group-fixture-checks

Conversation

@TomasVotruba

Copy link
Copy Markdown
Member

Merge 3 separate fixture-related matrix entries in code_analysis.yaml into one Fixture Checks job. Fewer separate GitHub checks, same coverage.

Before — 3 checks:

  • Check before/after test fixture on no-changes
  • Check fixture classes are different to nodes
  • Check no "*.php" files in rules Fixture directory

After — 1 check Fixture Checks:

-
    name: 'Fixture Checks'
    run: |
        php scripts/check-before-after-same-fixtures.php
        php scripts/avoid-short-node-names-in-fixtures.php
        php scripts/no-php-file-in-fixtures.php

Note: if branch protection lists the old 3 check names as required, update required checks to Fixture Checks.

@TomasVotruba
TomasVotruba merged commit 09a140b into main Aug 24, 2026
45 of 46 checks passed
@TomasVotruba
TomasVotruba deleted the tv-group-fixture-checks branch August 24, 2026 23:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant