-
Notifications
You must be signed in to change notification settings - Fork 28
Update github.com/spf13/cobra from v1.9.1 to v1.10.1 #3445
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
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -673,7 +673,7 @@ timeout-minutes: 1 | |
| # Deprecated: Use 'timeout-minutes' instead. Workflow timeout in minutes. Defaults | ||
| # to 15 minutes for agentic workflows. | ||
| # (optional) | ||
| timeout-minutes: 10 | ||
| timeout_minutes: 10 | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Nitpick: Documentation inconsistency - the example shows Why it matters: This documentation example could confuse users since GitHub Actions fields use hyphens, not underscores. The comment above correctly states "Use 'timeout-minutes' instead" but the example contradicts this. Suggested fix: Change to
|
||
|
|
||
| # Concurrency control to limit concurrent workflow runs (GitHub Actions standard | ||
| # field). Agentic workflows use enhanced concurrency management. | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The field name in the example should be 'timeout-minutes' not 'timeout_minutes'. According to the documentation on line 673, this field is deprecated and should use 'timeout-minutes' instead. However, since this is documenting the deprecated field, it should remain as 'timeout_minutes' to show the old syntax. The change appears to be unintentional - reverting from 'timeout-minutes' to 'timeout_minutes'.