Skip to content
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

Upgrade PHPCompatibility to 7.1.5 #45

Merged
merged 2 commits into from
Jan 19, 2018

Conversation

polothy
Copy link
Contributor

@polothy polothy commented Aug 23, 2017

Downloaded and dropped in the 7.1.5 release.

Release notes: https://github.com/wimg/PHPCompatibility/releases/tag/7.1.5

In particular, this release removes NewMagicClassConstant (or at least the bad parts) which would incorrectly warn about ::class usage.

@polothy
Copy link
Contributor Author

polothy commented Aug 24, 2017

Travis failures are due to running master on PHP5.6

@polothy
Copy link
Contributor Author

polothy commented Oct 4, 2017

@stronk7 Interested in this PR? Unsure if you saw it. No rush :)

@stronk7
Copy link
Member

stronk7 commented Jan 18, 2018

Oh @mrmark , sorry. I had this completely missed since 3.4 freeze, grrr. Looking now... and planning to do some work towards CONTRIB-6209 soon too. Thanks and excuse me!

@stronk7
Copy link
Member

stronk7 commented Jan 18, 2018

Amazing... just saw the cs_291_wip branch that I have here too.. anyway.. later...now looking to phpcompatibility...

@polothy
Copy link
Contributor Author

polothy commented Jan 18, 2018

Great, thanks!

@stronk7 stronk7 merged commit 0d32140 into moodlehq:master Jan 19, 2018
@stronk7
Copy link
Member

stronk7 commented Jan 19, 2018

Merged, thanks! About to tidy some tests and run them now...

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