Skip to content
This repository was archived by the owner on Apr 29, 2024. It is now read-only.

Comments

Fix exit code of parallel_rspec#40

Open
iGEL wants to merge 3 commits intosevos:masterfrom
vane-tech:master
Open

Fix exit code of parallel_rspec#40
iGEL wants to merge 3 commits intosevos:masterfrom
vane-tech:master

Conversation

@iGEL
Copy link
Collaborator

@iGEL iGEL commented Nov 11, 2016

Without English being required, $CHILD_STATUS in line 26 is
undefined and thus evaluates to nil, and nil.to_i is 0. So
parallel_rspec always returned 0 -> success, even when specs
failed.

Without `English` being required, `$CHILD_STATUS` in line 26 is
undefined and thus evaluates to `nil`, and `nil.to_i` is `0`. So
parallel_rspec always returned `0` -> success, even when specs
failed.
@sevos
Copy link
Owner

sevos commented Nov 11, 2016

do you think you can fix circleCI?

@sevos
Copy link
Owner

sevos commented Nov 11, 2016

feel free to push fixes. I can release the new version of gem to rubygems once we bump it ;)

@iGEL iGEL force-pushed the master branch 3 times, most recently from 7bbe2fd to 4940c56 Compare November 11, 2016 22:06
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants