Skip to content

Commit 39e62b8

Browse files
PKuhlmayclaude
andcommitted
[TASK] Wire up Rector CI job and exclude test fixtures from Rector scope
- rector.php: exclude Tests/Fixtures from Rector to avoid breaking CompatibilityTest (fixtures are reference files for the generator, not production code) - linters.yml: implement rectortest job with PHP 8.3 setup and dry-run check Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent 1aef1e2 commit 39e62b8

2 files changed

Lines changed: 14 additions & 7 deletions

File tree

.github/workflows/linters.yml

Lines changed: 13 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -15,17 +15,23 @@ on:
1515
# A workflow run is made up of one or more jobs that can run sequentially or in parallel
1616
jobs:
1717
rectortest:
18-
# The type of runner that the job will run on
1918
runs-on: ubuntu-latest
2019

21-
# Steps represent a sequence of tasks that will be executed as part of the job
2220
steps:
23-
# Checks-out your repository under $GITHUB_WORKSPACE, so your job can access it
24-
- uses: actions/checkout@v4
21+
- name: "Checkout code"
22+
uses: actions/checkout@v4
23+
24+
- name: "Setup PHP"
25+
uses: shivammathur/setup-php@v2
26+
with:
27+
php-version: 8.3
28+
coverage: none
29+
30+
- name: "Install dependencies"
31+
run: composer install --prefer-dist --no-progress --no-interaction
2532

26-
# Runs a single command using the runners shell
27-
- name: "Run a one-line script"
28-
run: echo This should run the rector test. This is not implemented yet
33+
- name: "Run Rector (dry-run)"
34+
run: .Build/bin/rector process --dry-run
2935

3036
sass-linter:
3137
# The type of runner that the job will run on

rector.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@
77

88
return RectorConfig::configure()
99
->withPaths([__DIR__ . '/Classes', __DIR__ . '/Tests'])
10+
->withSkip([__DIR__ . '/Tests/Fixtures'])
1011
->withSets([
1112
Typo3SetList::TYPO3_12,
1213
]);

0 commit comments

Comments
 (0)