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

Make the help text of load-into-counting's '-b' flag more verbose #857

Merged
merged 3 commits into from
Mar 1, 2015

Conversation

kdm9
Copy link
Contributor

@kdm9 kdm9 commented Feb 27, 2015

Hi all,

I confused myself this morning while trying to work out the effect of -b in load-into-counting. Admittedly I was having a bit of a slow morning, but I got confused by the --no-bigcounts (default=True) help message. Does default=True mean that it's off or on by default?

This PR explains the behavior more verbosely in the help string, hopefully proofing against coffee deprived users 😄.

Cheers,
K

@ctb
Copy link
Member

ctb commented Feb 28, 2015

LGTM. Pls make mergable at your convenience, and ping me when you do so :)

@kdm9 kdm9 force-pushed the fix/clearer-lic-nobigcount branch from 4dfcc58 to dd07b91 Compare March 1, 2015 05:07
@kdm9
Copy link
Contributor Author

kdm9 commented Mar 1, 2015

Rebased, so ping @ctb

ctb added a commit that referenced this pull request Mar 1, 2015
Make the help text of load-into-counting's '-b' flag more verbose
@ctb ctb merged commit 996493f into dib-lab:master Mar 1, 2015
@kdm9 kdm9 deleted the fix/clearer-lic-nobigcount branch March 3, 2015 03:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants