-
Notifications
You must be signed in to change notification settings - Fork 0
ci: auto-update the macros submodule via Dependabot #117
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 | ||||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| @@ -0,0 +1,16 @@ | ||||||||||||||||||||||
| # Keep git submodules up to date automatically. | ||||||||||||||||||||||
| # | ||||||||||||||||||||||
| # This template embeds the shared `macros` repo as a git submodule. Dependabot's | ||||||||||||||||||||||
| # `gitsubmodule` updater opens a PR whenever a submodule's default branch | ||||||||||||||||||||||
| # (currently `main`) advances, so neither this template nor the books generated | ||||||||||||||||||||||
| # from it have to bump the pointer by hand. See the ADR at d-morrison/macros | ||||||||||||||||||||||
| # (docs/adr/0001-macros-submodule-versioning.md) for why we track `main` rather | ||||||||||||||||||||||
| # than a pinned release tag. | ||||||||||||||||||||||
| version: 2 | ||||||||||||||||||||||
| updates: | ||||||||||||||||||||||
| - package-ecosystem: "gitsubmodule" | ||||||||||||||||||||||
| directory: "/" | ||||||||||||||||||||||
| schedule: | ||||||||||||||||||||||
| interval: "weekly" | ||||||||||||||||||||||
| commit-message: | ||||||||||||||||||||||
| prefix: "chore(submodule)" | ||||||||||||||||||||||
|
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. The
Suggested change
Note: the two |
||||||||||||||||||||||
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 repo's workflows use a number of third-party Actions at pinned major versions (
actions/checkout@v4,r-lib/actions/setup-r@v2,quarto-dev/quarto-actions/setup@v2, etc.). Now thatdependabot.ymlexists, adding agithub-actionsecosystem entry here is zero extra effort and keeps action pins current automatically:Not a blocker — just a natural complement to the submodule entry.
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.
Good idea, but I'm keeping this PR scoped to the submodule automation that was requested. A
github-actionsecosystem entry is a separate (also reasonable) kind of update that would start opening action-bump PRs across this template and every repo generated from it, so it's worth deciding on deliberately rather than bundling it in here. Easy to add as a follow-up if you'd like it — the snippet above is exactly right.Generated by Claude Code