Skip to content

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

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

ci: add Dependabot and auto-merge for hotdata SDK updates#7
eddietejeda merged 3 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
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: PR claims to add auto-merge for hotdata SDK updates, but no auto-merge workflow is included in the diff. Only the Dependabot config is added.
  • .github/dependabot.yml: Config has no allow: filter, so it will open PRs for every pip dependency (hotdata-runtime, hotdata, llama-index-core, pytest), not just hotdata as the title/description states.

Action Required

Either:

  1. Add the missing auto-merge workflow (gated on dependabot[bot] actor + dependabot/fetch-metadata to verify the updated package is in the hotdata SDK), and add an allow: filter to scope Dependabot to the hotdata packages; or
  2. Update the PR title and description to accurately reflect that this change only enables daily Dependabot for all pip dependencies, with no auto-merge.

@eddietejeda eddietejeda merged commit 0c46e0e into main Jun 1, 2026
3 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