Skip to content

Fix --size_format argument definition. - #130

Merged
schlessera merged 7 commits into
masterfrom
fix-size-format-arg
Jan 29, 2019
Merged

schlessera merged 7 commits into
masterfrom
fix-size-format-arg

Conversation

@danielbachhuber

Copy link
Copy Markdown
Member

It should be --size_format=<format> because it's never a flag. From #16.

Also adds --human-readable to README docs. From #124.

It should be `--size_format=<format>` because it's never a flag.

Also adds `--human-readable` to README docs.
@danielbachhuber danielbachhuber added this to the 2.0.2 milestone Jan 24, 2019
@danielbachhuber
danielbachhuber requested a review from a team January 24, 2019 14:11
@danielbachhuber danielbachhuber added command:db-size Related to 'db size' command scope:documentation Related to documentation labels Jan 24, 2019
@danielbachhuber

Copy link
Copy Markdown
Member Author

Hm. Guess I should get these tests passing locally before I push my commits...

@danielbachhuber

Copy link
Copy Markdown
Member Author

@schlessera Any ideas why this might be failing in Travis? It passes for me locally.

@schlessera

Copy link
Copy Markdown
Member

@danielbachhuber It succeeded on my local system too, so I cleared the Travis caches and restarted the build, and it seems to have solved the issue so far.

@schlessera
schlessera merged commit 0c2f1e8 into master Jan 29, 2019
@delete-merged-branch
delete-merged-branch Bot deleted the fix-size-format-arg branch January 29, 2019 16:43
@danielbachhuber

Copy link
Copy Markdown
Member Author

Interesting! Thanks :)

danielbachhuber pushed a commit that referenced this pull request Nov 18, 2022
Fix `--size_format` argument definition.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

command:db-size Related to 'db size' command scope:documentation Related to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants