Skip to content

allow Nette 3.0#1323

Merged
TomasVotruba merged 2 commits intorectorphp:masterfrom
mimmi20:master
Apr 16, 2019
Merged

allow Nette 3.0#1323
TomasVotruba merged 2 commits intorectorphp:masterfrom
mimmi20:master

Conversation

@mimmi20
Copy link
Copy Markdown
Contributor

@mimmi20 mimmi20 commented Apr 16, 2019

I wanted to test this tool, but I require Nette 3.0.

By the way, Is it normal that your PHPUnit coverage tests require more than 25 minutes on travis ci?

@TomasVotruba TomasVotruba merged commit de0d918 into rectorphp:master Apr 16, 2019
@TomasVotruba
Copy link
Copy Markdown
Member

TomasVotruba commented Apr 16, 2019

Thanks 👍

It usually takes 4-5 mins.

In this PR it's 6 mins:

image

Where is 25 mins? That would mean something is wrong

@mimmi20
Copy link
Copy Markdown
Contributor Author

mimmi20 commented Apr 16, 2019

image

@TomasVotruba
Copy link
Copy Markdown
Member

Could you share link?

@sanmai
Copy link
Copy Markdown
Contributor

sanmai commented Apr 16, 2019

This got to be a glitch. Like, sometimes Composer can’t do its thing on time., or the like. Should be in the logs.

@mimmi20
Copy link
Copy Markdown
Contributor Author

mimmi20 commented Apr 16, 2019

@TomasVotruba
Copy link
Copy Markdown
Member

TomasVotruba commented Apr 16, 2019

Yes I can, for open source projects it's always public.

Probably some glitch in the Matrix. If it happens again, let me know

TomasVotruba added a commit that referenced this pull request Nov 28, 2021
rectorphp/rector-src@3f6f706 [DeadCode] Add RemoveDeadContinueRector (#1323)
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.

3 participants