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

Add option to preserve ids that are provided as arguments with --prepare-jobs #836

Merged
merged 2 commits into from
Feb 6, 2025

Conversation

nille02
Copy link
Contributor

@nille02 nille02 commented Jan 7, 2025

If you try to combine --prepare-jobs and running Job IDs in one command the ids got ignore since prepare_jobs overrides the set.

With this small change we use a set instead of a list. we use a set and merge them later together to preserve other scheduled jobs.

With this we can use for example urlwatch 1 2 3 4 --prepare-jobs

…are-jobs

If you try to combine --prepare-jobs and running Job IDs in one command the ids got ignores since prepare_jobs overrides the set.

With this small change we use a set instead of a list we use a set and merge them later together to preserve other scheduled jobs.
Replace a "literal" with a function call.

Co-authored-by: Thomas Perl <[email protected]>
@nille02 nille02 requested a review from thp January 18, 2025 10:44
@thp thp merged commit 30f143b into thp:master Feb 6, 2025
5 checks passed
@nille02 nille02 deleted the prepare-jobs-extended branch February 8, 2025 18:07
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