Skip to content

Fix PHPStan failures#286

Merged
alcaeus merged 1 commit intodoctrine:masterfrom
sunspikes:master
Oct 1, 2019
Merged

Fix PHPStan failures#286
alcaeus merged 1 commit intodoctrine:masterfrom
sunspikes:master

Conversation

@sunspikes
Copy link
Copy Markdown

Addressing #285

SenseException
SenseException previously approved these changes Oct 1, 2019
@alcaeus alcaeus changed the title Remove PHP 7.4 from allow_failures Fix PHPStan failures Oct 1, 2019
@alcaeus
Copy link
Copy Markdown
Member

alcaeus commented Oct 1, 2019

@sunspikes merging up 1.8 fixed PHPUnit on PHP 7.4. However, travis-ci was still broken, which you fixed here. I rebased your branch on top of the new master and kept only the travis-ci relevant changes. This way you still get credit for your Hacktoberfest contribution. Thank you very much!

@alcaeus alcaeus merged commit 53db8bf into doctrine:master Oct 1, 2019
@alcaeus
Copy link
Copy Markdown
Member

alcaeus commented Oct 1, 2019

Merging here - I'll take a look at scrutiniser-ci some other time.

@alcaeus alcaeus added this to the v2.0.0 milestone Oct 1, 2019
@alcaeus alcaeus self-assigned this Oct 1, 2019
@sunspikes
Copy link
Copy Markdown
Author

@alcaeus thank you so much!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants