-
Notifications
You must be signed in to change notification settings - Fork 396
Trigger sync back script automatically #3140
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
Conversation
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.
Pull Request Overview
This PR streamlines the Dependabot configuration by consolidating GitHub Actions monitoring into a single entry that covers multiple directories, and automatically adds the "Rebuild" label to GitHub Actions updates to trigger the sync back script.
- Consolidates two separate GitHub Actions package ecosystem entries into one with multiple directories
- Adds automatic "Rebuild" label to GitHub Actions updates to trigger sync back automation
- Simplifies the configuration by removing the separate setup-swift specific entry
directories: | ||
- "/" | ||
- "/.github/actions" # All subdirectories outside of "/.github/workflows" must be explicitly included. |
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 directories
field is not a valid Dependabot configuration option for GitHub Actions. The correct field name is directory
(singular), and it only accepts a single directory path, not an array. To monitor multiple directories, you need separate package-ecosystem entries.
Copilot uses AI. Check for mistakes.
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.
I vaguely remember that this is/was the case for github-actions
as Copilot says, but I can't find any documentation which confirms this. I am happy to approve and merge this, which should trigger a validation of the dependabot.yml
file. If it doesn't work, then we can easily patch it up afterwards.
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.
directories: | ||
- "/" | ||
- "/.github/actions" # All subdirectories outside of "/.github/workflows" must be explicitly included. |
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.
I vaguely remember that this is/was the case for github-actions
as Copilot says, but I can't find any documentation which confirms this. I am happy to approve and merge this, which should trigger a validation of the dependabot.yml
file. If it doesn't work, then we can easily patch it up afterwards.
Also enable Dependabot updates for the other directories in
.github/actions
.Risk assessment
For internal use only. Please select the risk level of this change:
Merge / deployment checklist