Skip to content

Conversation

@TomasVotruba
Copy link
Member

No description provided.

@TomasVotruba TomasVotruba force-pushed the tv-before-after-ci branch 2 times, most recently from 6e24025 to 835f6ff Compare August 11, 2025 06:37

-
name: 'Check before/after test fixture on no-changes'
run: php bin/check-before-after-same-fixtures.php
Copy link
Member Author

@TomasVotruba TomasVotruba Aug 11, 2025

Choose a reason for hiding this comment

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

@samsonasik This might help with code reviews 👍

Copy link
Member

Choose a reason for hiding this comment

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

@TomasVotruba the files seems commited on scoped vendor, should it be removed before scoped?

rectorphp/rector@d2d451e#diff-6a943cc1e536167118f5fb8160e164027ae139c4986d1e324a1faabbd1f39b36

Copy link
Member Author

Choose a reason for hiding this comment

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

👍 Indeed, should be removed

Copy link
Member

Choose a reason for hiding this comment

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

@TomasVotruba TomasVotruba force-pushed the tv-before-after-ci branch 5 times, most recently from 60a6093 to be61386 Compare August 11, 2025 07:01
@TomasVotruba TomasVotruba changed the title [ci] check extra after part in fixture, that show no change [ci] check extra after part in fixture, that show no change + check no PHP files in /Fixture dirs Aug 11, 2025
@TomasVotruba TomasVotruba enabled auto-merge (squash) August 11, 2025 07:24
@TomasVotruba TomasVotruba merged commit 9ab4f86 into main Aug 11, 2025
49 checks passed
@TomasVotruba TomasVotruba deleted the tv-before-after-ci branch August 11, 2025 07:24
@github-actions
Copy link
Contributor

This pull request has been automatically locked because it has been closed for 150 days. Please open a new PR if you want to continue the work.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 11, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants