Skip to content

tests: remove fragile pre-release tests#752

Merged
blizzz merged 5 commits intomasterfrom
fix/prerelease-tests
May 7, 2026
Merged

tests: remove fragile pre-release tests#752
blizzz merged 5 commits intomasterfrom
fix/prerelease-tests

Conversation

@blizzz
Copy link
Copy Markdown
Member

@blizzz blizzz commented May 6, 2026

the rebuild also bumps the version from 33 to 34 beta 😏

@blizzz blizzz requested review from Altahrim and skjnldsv as code owners May 6, 2026 17:18
@blizzz blizzz force-pushed the fix/prerelease-tests branch 4 times, most recently from c06baca to 7089031 Compare May 6, 2026 17:50
blizzz added 5 commits May 6, 2026 19:58
as we do not keep them anymore on the download server

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
@blizzz blizzz force-pushed the fix/prerelease-tests branch from 7089031 to cd1f57a Compare May 6, 2026 17:58
Comment thread composer.json
},
"scripts": {
"box": "box compile -c box.json",
"box": "@php box compile -c box.json",
Copy link
Copy Markdown
Collaborator

@Altahrim Altahrim May 7, 2026

Choose a reason for hiding this comment

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

It forces box to use the same PHP version than the one used to launch composer?

Copy link
Copy Markdown
Member Author

@blizzz blizzz May 7, 2026

Choose a reason for hiding this comment

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

The same version as composer is run with, yes. Cf. https://getcomposer.org/doc/articles/scripts.md#executing-php-scripts

@blizzz blizzz merged commit dbd620a into master May 7, 2026
21 checks passed
@blizzz blizzz deleted the fix/prerelease-tests branch May 7, 2026 09:38
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