Skip to content

Added composer rector-ci to workflow#2714

Merged
TomasVotruba merged 6 commits intorectorphp:masterfrom
jeroensmit:rector-ci
Jan 19, 2020
Merged

Added composer rector-ci to workflow#2714
TomasVotruba merged 6 commits intorectorphp:masterfrom
jeroensmit:rector-ci

Conversation

@jeroensmit
Copy link
Copy Markdown
Contributor

First merge #2712 and #2713

@TomasVotruba
Copy link
Copy Markdown
Member

Both PRs merged. ECS is failing

php-version: 7.3
coverage: none # disable xdebug, pcov
- run: composer install --no-progress
- run: composer rector-ci
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

You remove the rector job above. It's duplicated now and only with --dry-run it actually fails the CI

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I'll handle it

@TomasVotruba TomasVotruba self-requested a review January 19, 2020 21:33
@TomasVotruba TomasVotruba merged commit 8d22139 into rectorphp:master Jan 19, 2020
@TomasVotruba
Copy link
Copy Markdown
Member

Great work 👍

TomasVotruba added a commit that referenced this pull request Jul 26, 2022
rectorphp/rector-src@32834d0 [DeadCode] Handle crash on negation class const fetch with static:: and parent:: on RemoveDeadZeroAndOneOperationRector (#2714)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants