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

Use nopt to parse command-line options #68

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

Conversation

hsandt
Copy link

@hsandt hsandt commented Aug 4, 2019

In order to add new options like "--newline-separator" in the future, without messing up with the existing options, I switched option parsing to nopt, which should make it easier to detect options anywhere in the arguments (currently, anything passed after --code, --file or --ast would be considered a "snippet" as part of the positional arguments).

This change is mostly retrocompatible for correct usage, supporting a few new cases like luamin "a = 5" --code (--code can be put after the positional arguments), and detecting compact forms like luamin -ch for luamin -c -h (obviously a bad example since it will ignore -c and just print help, but will be useful when we add more options, compatible with -c, -f and -a).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant