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

Fix crash on initial run when no configuration exists #3

Open
wants to merge 2 commits into
base: master
from

Conversation

Projects
None yet
3 participants
@nilbus

nilbus commented Mar 2, 2015

jutebag.commands used to be an object with command names as the keys, until
e69f7df changed jutebag.commands from an
object to a sorted array.

Use an appropriate lookup, now that jutebag.commands is an array.

Also, make bin/jutebag executable, so it can be run directly, without
explicitly running node while developing.

nilbus added some commits Mar 2, 2015

Fix crash on initial run
jutebag.commands used to be an object with command names as the keys,
until e69f7df changed jutebag.commands
from an object to a sorted array.

Use an appropriate lookup, now that jutebag.commands is an array.

Fixes #1

@nilbus nilbus referenced this pull request Mar 2, 2015

Open

jb init crashes #1

@shedali

This comment has been minimized.

Show comment
Hide comment
@shedali

shedali Apr 3, 2015

this works for me, thanks 👍

shedali commented Apr 3, 2015

this works for me, thanks 👍

@MaxPleaner

This comment has been minimized.

Show comment
Hide comment
@MaxPleaner

MaxPleaner Oct 31, 2015

Thanks this pull request fixed it for me.

MaxPleaner commented Oct 31, 2015

Thanks this pull request fixed it for me.

@nilbus

This comment has been minimized.

Show comment
Hide comment
@nilbus

nilbus Oct 31, 2015

@akoenig What do you think about merging and releasing this fix, André?

nilbus commented Oct 31, 2015

@akoenig What do you think about merging and releasing this fix, André?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment