Skip to content

ci: add Dependabot and auto-merge for hotdata SDK updates#18

Merged
eddietejeda merged 4 commits into
mainfrom
ci/dependabot-automerge
Jun 1, 2026
Merged

ci: add Dependabot and auto-merge for hotdata SDK updates#18
eddietejeda merged 4 commits into
mainfrom
ci/dependabot-automerge

Conversation

@eddietejeda
Copy link
Copy Markdown
Contributor

@eddietejeda eddietejeda commented Jun 1, 2026

Adds daily Dependabot checks scoped to hotdata and hotdata-runtime packages, and an auto-merge workflow that squash-merges those PRs once all checks pass.

  • dependabot.yml: watches the correct package ecosystem (uv where uv sync --locked is used in CI, pip otherwise), scoped via allow: to hotdata packages only
  • dependabot-automerge.yml: enables squash auto-merge on any PR opened by dependabot[bot]

Comment thread .github/dependabot.yml
Comment thread .github/dependabot.yml Outdated
Copy link
Copy Markdown

@claude claude Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review

Blocking Issues

  • .github/dependabot.yml:6 — PR description promises auto-merge after checks pass, but no auto-merge workflow (or other mechanism) is included in the diff.
  • .github/dependabot.yml:3 — config tracks all pip dependencies, not just hotdata as the description states. Needs an allow: filter or a description update.

Action Required

  • Add a workflow that performs the auto-merge (e.g. dependabot/fetch-metadata + gh pr merge --auto --squash, scoped to hotdata), or amend the PR description to clarify how auto-merge is being configured outside this PR.
  • Decide whether Dependabot should track only hotdata (add allow:) or all pip deps (revise description). Make the config and description agree.

Comment thread .github/dependabot.yml Outdated
claude[bot]
claude Bot previously approved these changes Jun 1, 2026
Copy link
Copy Markdown

@claude claude Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Prior review feedback addressed: auto-merge workflow added, ecosystem switched to uv with allow: scoped to hotdata, and the dead hotdata-runtime entry removed.

@eddietejeda eddietejeda merged commit 99a8bc7 into main Jun 1, 2026
4 checks passed
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.

1 participant