Skip to content

syntax was updated to PHP 5.6 (::class string, short array syntax etc.) #169

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

Merged
merged 3 commits into from
Jan 21, 2017
Merged

syntax was updated to PHP 5.6 (::class string, short array syntax etc.) #169

merged 3 commits into from
Jan 21, 2017

Conversation

pavelvondrasek
Copy link
Contributor

was added PHPUnit to dev-dependencies

tests were updated to remove deprecations (please double-check it!!!)

  • BONUS: was added PHPStan to dev-dependencies it can help us to make better code and in future we can run it before tests to analyze code :)

And finally joke: I think that CndParserTest is here (in this package) so that i made this refactoring I wan't to mark it as skipped in the end but I was very surprised :-D :-D

was added PHPUnit to dev-dependencies

tests were updated to remove deprecations (please double-check it!!!)

+ BONUS: was added PHPStan to dev-dependencies it can help us to make better code and in future we can run it before tests to analyze code :)

And finally joke: I think that CndParserTest is here (in this package) so that i made this refactoring I wan't to mark it as skipped in the end but I was very surprised :-D :-D
@dbu dbu merged commit a4edcc0 into phpcr:master Jan 21, 2017
@dbu
Copy link
Member

dbu commented Jan 21, 2017

awsome, thanks!
yeah lets add the test skip separately. we want to revert that when the bug is fixed. please check for php 7 and only skip if 7

@pavelvondrasek
Copy link
Contributor Author

Ok, thank you too. I'm working on it in PHPCR-api-tests I will do that :-)

What do you think about increase min PHP version to 5.6 in PHPCR-api-tests? I would like to add phpunit 5.7 and remove deprecations :)

@dbu
Copy link
Member

dbu commented Jan 21, 2017 via email

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