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

Allow arguments to the subprocess #6

Merged
merged 1 commit into from
Aug 10, 2015
Merged

Allow arguments to the subprocess #6

merged 1 commit into from
Aug 10, 2015

Conversation

jturnbull
Copy link

Use argparse's REMAINDER function to pass everything (including arguments) to the subprocess

Relates to commit 75ade6d

Use argparse's REMAINDER function to pass everything (including arguments) to the subprocess

Relates to commit 75ade6d
@hltbra
Copy link

hltbra commented Aug 10, 2015

I was doing this:

cron-sentry -- myprogram --arg1 value1 --arg2 value2

But this is so much better. Thanks! I will merge today.

@hltbra
Copy link

hltbra commented Aug 10, 2015

This PR as-is will break my cron jobs, because the old way isn't valid anymore. I will merge it and change it to make it backwards compatible.

@hltbra hltbra merged commit 86b1ef9 into Yipit:master Aug 10, 2015
@jturnbull jturnbull deleted the argparse_remainder branch August 10, 2015 19:21
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