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

Simplify Cli Structure #87

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

arcanearronax
Copy link

This change removes the need to parse arguments from sys.argv by using Click in order to handle the command line arguments. The flags that can be used have been modified a bit to support this change.

  • --ignore/-i This is used to identify which file extensions to ignore when sorting.
  • --sched/-s This is used to schedule sorting.
  • --desched/-d This is used to deschedule sorting.
  • --undosort/-u This is used to undo a sort.
  • -h This is used to display the different command line arguments available and usage.

One notable change to usage is that the extensions to ignore during sorting can be passed with or without the preceding period, multiple --ignore/-i flags can be used to pass multiple extensions (a comma delimited string of extensions can still be passed), and the parsing of extensions is handled in a more pythonic way.

…rguments. This includes modifying the flags used to call various processes and how file extensions are provided. The to_clean dict in Cli.cli was modified to rely on more pythonic comprehension, rather than for loops. The requirements file was updated to include the Click module as well.
…passed via sys.argv. The flags that can be used are --undosort/-u, --sched/-s, --desched/-d, --ignore/-i, and -h. The --ignore/-i flag is used to identify which file extensions to ignore when sorting. The --ignore/-i flag can be passed multiple times, once for each extension, or parse a comma delimited string of extensions. The extensions can be passed with or without the preceding period.
Merging existing changes from master into click_integration for consistency.
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.

None yet

1 participant