Skip to content

Conversation

@skurfuerst
Copy link
Member

@skurfuerst skurfuerst commented Aug 31, 2025

This helps me with a version conflict I am currently encountering, and should have no negative side-effects. I want to try out PestPHP with Neos, especially Browser Testing: https://pestphp.com/docs/pest-v4-is-here-now-with-browser-testing#content-pest-v4-is-here-now-with-browser-testing

… the CR (only for developing it)

This helps me with a version conflict I am currently encountering
@dlubitz
Copy link
Contributor

dlubitz commented Aug 31, 2025

This will not work, as the test result also shows. Because dev-dependencies are only installed in the distribution package and not of dependent packages.

@kitsunet
Copy link
Member

kitsunet commented Sep 1, 2025

The quintessential crux for how to deal with dev dependencies, however I think we just could add paratest as dev dependency of the neos-development-distribution

@mhsdesign
Copy link
Member

Jup i feared that it will not stay with my fix #5592 but that the idea of having a test package inside the dev collection will always be funny with the dependencies.

We definitely need a working way to define the dev dependencies in the dev collection for the outside installed distribution ... maybe some small script that copies the entries to the base distribution before the tests are run? But for now just hardcode them there i agree.

@kitsunet
Copy link
Member

kitsunet commented Sep 1, 2025

neos/neos-development-distribution#112

I guess if we merge the above and then rerun the jobs it should wokr?! then we can merge this

@nezaniel nezaniel self-requested a review October 23, 2025 12:58
Copy link
Member

@nezaniel nezaniel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

makes sense to me

@mhsdesign mhsdesign merged commit 4279c9f into 9.0 Oct 23, 2025
22 of 26 checks passed
@mhsdesign mhsdesign deleted the bugfix-paratest-as-require-dev branch October 23, 2025 17:25
@skurfuerst
Copy link
Member Author

Thanks everybody ❤️

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.

6 participants