wikimedia / composer-merge-plugin

Merge one or more additional composer.json files at Composer runtime
MIT License
923 stars 160 forks source link

Propagate exit code when running post-install hook #244

Open harnash opened 1 year ago

harnash commented 1 year ago

This is related to #180.

We have noticed that when post-install hook is executed composer-merge-plugin executed the whole process again and when that fails it just logs an error and does not propagate exit status code to the parent process causing composer to return exit status code 0 even when the process clearly failed. This change should propagate exit status code and will help preventing broken builds in CI/CD environments when tooling reiles on exit status codes to determine if the command finished successfully or not.