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

BREAKING: Add retryable arg to stage decorator #23

Open
jmgate opened this issue Jun 25, 2024 · 0 comments
Open

BREAKING: Add retryable arg to stage decorator #23

jmgate opened this issue Jun 25, 2024 · 0 comments
Assignees
Labels
Type: Refactor Refactoring existing code to improve long-term sustainability.

Comments

@jmgate
Copy link
Collaborator

jmgate commented Jun 25, 2024

It's a bit of a learning curve for those trying to create bare-bones staged scripts to see all those retry command line arguments automatically populated when they haven't designed their stages to be retryable. A better user experience would be to add a retryable flag to the stage() decorator and default it to False. Only if True do all those retry arguments get generated.

Note: This will be a breaking change.

@jmgate jmgate added the Type: Refactor Refactoring existing code to improve long-term sustainability. label Jun 25, 2024
@jmgate jmgate changed the title Add retryable arg to stage decorator BREAKING: Add retryable arg to stage decorator Jun 25, 2024
@jmgate jmgate added Stage: Development Someone's working on this currently. and removed Stage: Development Someone's working on this currently. labels Jul 3, 2024
@jmgate jmgate self-assigned this Sep 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Refactor Refactoring existing code to improve long-term sustainability.
Projects
None yet
Development

No branches or pull requests

1 participant