Skip to content

Conversation

@rosier
Copy link
Contributor

@rosier rosier commented Aug 7, 2025

I did some fixes and tweaks for the demo install test.

I did multiple commit to make it easier to review

tools: composer
coverage: none
extensions: none, ctype, dom, iconv, intl, mbstring, pdo_sqlite, simplexml, tokenizer, xml, xmlwriter
php-version: latest
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Since the above test with symfony new also does use composer create-project internally I opted for a different PHP version for this job

@javiereguiluz javiereguiluz merged commit 92d9dd5 into symfony:main Aug 14, 2025
8 checks passed
@javiereguiluz
Copy link
Member

Jan, these are great changes and improvements, as always. Thanks!

While merging I removed the rm -v .env.local line that you introduced to force an error and chekc this. I'll readd it in a PR that I'm submitting next to change the issue creation a bit.

@rosier rosier deleted the ci-install-test branch August 30, 2025 11:10
javiereguiluz added a commit that referenced this pull request Sep 9, 2025
…(javiereguiluz)

This PR was squashed before being merged into the main branch.

Discussion
----------

[CI] Update the action that creates issues on e2e errors

This continues #1590 to update the action that creates issues on CI errors. The `rm -v ...` line was added on purpose to force an error to check that this is working.

This was done entirely with AI.

Commits
-------

675dddc [CI] Update the action that creates issues on e2e errors
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