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

Improve transmission upload script #3

Merged
merged 3 commits into from
Oct 29, 2017

Conversation

mutantmonkey
Copy link
Member

  • Switch to argparse because it's more popular and comfortable for me than docopt.
  • Add --limit-tracker argument to limit the list of torrents to only those that use the specified tracker.
  • Use a default --transmission-port of 9091 since this will be the case for most configurations.

* Switch to argparse
* Add optional --limit-tracker argument to limit the list of returned
  torrents by the tracker they use.
@matthazinski
Copy link
Member

Thanks for adding smuggler auth support.

Re: the switch to argparse, the option names are different which makes this is a breaking change. Can you add -u/-p/-P/--server as aliases to the long-form commands?

Copy link
Member

@matthazinski matthazinski left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Need to add aliases for short-form arguments

@matthazinski matthazinski merged commit c34f893 into master Oct 29, 2017
@matthazinski matthazinski deleted the transmission_upload_improvements branch October 29, 2017 04:02
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.

2 participants