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

Comments should not make exit code incorrect. Fixes #92 #168

Merged
merged 1 commit into from
Aug 3, 2015
Merged

Comments should not make exit code incorrect. Fixes #92 #168

merged 1 commit into from
Aug 3, 2015

Conversation

geraintwhite
Copy link

PR #94 wasn't merged in, so creating a new PR

@ljharb
Copy link
Collaborator

ljharb commented Aug 2, 2015

v2.13.4 does have that commit merged in, in 17894ca (note its parent), but you're right that this never made it into master, and thus v3.x or v4.x.

After this gets merged in, it should probably also be backported on top of v3.0.3, v3.1.0, v3.2.0, v3.3.0, v3.4.0, v3.5.1, and v3.6.0 - and all of them bumped with a patch release, before publishing this change as v4.0.2. @substack / @Raynos, sound ok?

@Raynos
Copy link
Collaborator

Raynos commented Aug 3, 2015

back porting is amount of effort.

Let's get it into master and cut a new patch.

Raynos added a commit that referenced this pull request Aug 3, 2015
Comments should not make exit code incorrect. Fixes #92
@Raynos Raynos merged commit 0ded7d6 into tape-testing:master Aug 3, 2015
@geraintwhite geraintwhite deleted the issue-92 branch August 3, 2015 07:18
@Raynos
Copy link
Collaborator

Raynos commented Aug 3, 2015

Published 4.0.2

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.

4 participants