Skip to content
This repository has been archived by the owner on Jun 9, 2019. It is now read-only.

Fix #17 - fail_now should result in exit code 1 #18

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

johscheuer
Copy link

This PR ensures that fail_now and all other assertions that use assert.AssertionError results in an exit code 1

@jsleeio
Copy link

jsleeio commented Feb 19, 2018

looks ok to me, though I haven't personally tried the function you're referring to

thanks for sharing!

@vgramer
Copy link

vgramer commented Aug 28, 2018

@garethr can you merge and release this PR ?

@yastij
Copy link

yastij commented Aug 28, 2018

@garethr - can you release this patch when you have some bandwidth ? Thanks !

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.

4 participants