Skip to content

9 set default time limit#10

Merged
aplowman merged 2 commits intomainfrom
9-set-default-time-limit
May 15, 2025
Merged

9 set default time limit#10
aplowman merged 2 commits intomainfrom
9-set-default-time-limit

Conversation

@gcapes
Copy link
Collaborator

@gcapes gcapes commented May 15, 2025

Fix #9

gcapes added 2 commits May 15, 2025 10:29
This doesn't end up as a comment when importing the config,
so it isn't useful to the user.
@gcapes gcapes requested a review from aplowman May 15, 2025 10:19
@gcapes
Copy link
Collaborator Author

gcapes commented May 15, 2025

I'm not sure about all the yamllint errors on the config file. A list in yaml doesn't need the bullet/dash to be indented. It can be, but it's valid yaml without indentation.

As for the linting warning fromthe workflow file...? 🤷

@gcapes gcapes marked this pull request as ready for review May 15, 2025 10:24
@aplowman aplowman merged commit 21bae83 into main May 15, 2025
1 check failed
@gcapes gcapes deleted the 9-set-default-time-limit branch October 23, 2025 12:47
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.

Set default time limit

2 participants

Comments