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 mainAug 24, 2026
45 of 46 checks passed
@TomasVotruba
TomasVotruba deleted the tv-group-fixture-checks branch August 24, 2026 23:44
Sign up for freeto 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

@TomasVotruba