Skip to content
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

Revert to older automatic releases #9061

Merged

Conversation

greg0ire
Copy link
Member

@greg0ire greg0ire commented Oct 3, 2021

1.12.0 and up comes with a migration to azjezz/psl that makes it
impossible to troubleshoot issues with external commands such as git push

Example of failing build: https://github.com/doctrine/orm/runs/3782157355?check_suite_focus=true#step:4:18
Migration to PSL: laminas/automatic-releases@6ad991b
Issue on automatic-releases: laminas/automatic-releases#166

1.11.1 uses symfony/process, and unless output is disabled, it should display the output: https://github.com/symfony/process/blob/38f26c7d6ed535217ea393e05634cb0b244a1967/Exception/ProcessFailedException.php#L38-L43

1.12.0 and up comes with a migration to azjezz/psl that makes it
impossible to troubleshoot issues with external commands such as git push
@greg0ire greg0ire added the CI label Oct 3, 2021
@greg0ire greg0ire changed the base branch from 2.9.x to 2.10.x October 3, 2021 12:50
@greg0ire greg0ire merged commit 6d16147 into doctrine:2.10.x Oct 3, 2021
@greg0ire greg0ire deleted the revert-to-older-automatic-releases branch October 3, 2021 13:06
@greg0ire
Copy link
Member Author

greg0ire commented Oct 3, 2021

Looks like this does not help: https://github.com/doctrine/orm/runs/3782330485?check_suite_focus=true… psl is still in use…
The first mistake I did is PR'ing 2.10.x, but I backported to 2.9.x and the output shows 1.11.1. But it also shows this: ghcr.io/laminas/automatic-releases:1

@greg0ire
Copy link
Member Author

greg0ire commented Oct 3, 2021

I found a workaround: ddcea63

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant