Skip to content

CN-411 Fix exit code for parallel execution#947

Merged
antonmedv merged 1 commit into
deployphp:3.xfrom
ecentria:CN-411-fix-deployer-exit-code-in-case-of
Jan 16, 2017
Merged

CN-411 Fix exit code for parallel execution#947
antonmedv merged 1 commit into
deployphp:3.xfrom
ecentria:CN-411-fix-deployer-exit-code-in-case-of

Conversation

@aheart
Copy link
Copy Markdown

@aheart aheart commented Jan 6, 2017

Q A
Bug fix? Yes
New feature? No
BC breaks? No
Deprecations? No
Fixed tickets N/A

Backporting fix from 4.0
See 3e40fcd#diff-ecc506bcfd58cb6526cd2db33d5121cdR195

@antonmedv
Copy link
Copy Markdown
Member

Why you just can upgrade to 4.0 version?

@aheart
Copy link
Copy Markdown
Author

aheart commented Jan 6, 2017

One of our build servers is stuck with PHP 5.4 and will stay that way for an unknown period of time.
Everyone who is in a similar position could benefit from this, but If you would rather not invest time into 3.x I completely understand.
We can use our fork until an upgrade to 4.x is possible.

Thank you for taking the time to review this PR.

@antonmedv antonmedv merged commit d6f6310 into deployphp:3.x Jan 16, 2017
@antonmedv
Copy link
Copy Markdown
Member

@aheart can you use it via composer installation?

@aheart
Copy link
Copy Markdown
Author

aheart commented Jan 16, 2017

@Elfet No, but I've created a phar using your build script. Works well, thank you.

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