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

exits with success status when no COMMAND argument is given #32

Open
Akuli opened this issue Jan 4, 2019 · 2 comments
Open

exits with success status when no COMMAND argument is given #32

Akuli opened this issue Jan 4, 2019 · 2 comments
Assignees

Comments

@Akuli
Copy link

Akuli commented Jan 4, 2019

from manly.py:

    if not len(args.command):
        print_err(
            "manly: missing COMMAND\nTry 'manly --help' for more information.",
        )
        sys.exit(0)

many programs use 2 for invalid usage, argparse does it too

@Summertime
Copy link
Collaborator

man returns 1 for missing arg, which is probably what we should follow

@Summertime
Copy link
Collaborator

Fixed in 4a6ad1f, need to follow up with making tests, ! manly or manly ; [[ $? -eq 1 ]] should be best

@Summertime Summertime self-assigned this Jan 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants