-
-
Notifications
You must be signed in to change notification settings - Fork 32
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
Require additional and updated testing packages for the CR test suite #93
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fine by reading, but still have some questions ^^ - i guess if ci fails we need to revert xD
"symfony/css-selector": "^2.0", | ||
"vimeo/psalm": "~4.9.3", | ||
"vimeo/psalm": "^5.0", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
are we sure about this? - i the code compliant?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't be a problem (runs with phpunit 10 locally), but even if this fails for some reason it should be fixed asap in the dev collection
@@ -40,9 +40,10 @@ | |||
"neos/setup": "@dev", | |||
"flowpack/neos-frontendlogin": "@dev", | |||
"mikey179/vfsstream": "^1.6", | |||
"phpunit/phpunit": "^9.0", | |||
"behat/behat": "^3.0", | |||
"phpunit/phpunit": "^10.0", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
dont we have like code that needs to be adjusted? ^^
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
see above
No description provided.