-
-
Notifications
You must be signed in to change notification settings - Fork 432
[ci] check extra after part in fixture, that show no change + check no PHP files in /Fixture dirs #7134
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
6e24025 to
835f6ff
Compare
|
|
||
| - | ||
| name: 'Check before/after test fixture on no-changes' | ||
| run: php bin/check-before-after-same-fixtures.php |
There was a problem hiding this comment.
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 👍
There was a problem hiding this comment.
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
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 Indeed, should be removed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, I removed at PR:
60a6093 to
be61386
Compare
be61386 to
cceab77
Compare
|
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. |
No description provided.