Skip to content

fix(dependabot): merge helm entries to fix overlapping dirs#146

Merged
adamancini merged 1 commit intomainfrom
kriscoleman/fix-dependabot-dirs
Mar 31, 2026
Merged

fix(dependabot): merge helm entries to fix overlapping dirs#146
adamancini merged 1 commit intomainfrom
kriscoleman/fix-dependabot-dirs

Conversation

@kriscoleman
Copy link
Copy Markdown
Member

@kriscoleman kriscoleman commented Mar 31, 2026

Summary

  • Dependabot requires unique (package-ecosystem, directory) pairs — two helm entries with the same directories causes: "Update configs must have a unique combination of 'package-ecosystem', 'directory', and 'target-branch'. Ecosystem 'helm' has overlapping directories."
  • Consolidates into a single helm entry with two groups using patterns / exclude-patterns to separate Replicated SDK updates from everything else
  • Both groups now run weekly (Dependabot doesn't support per-group schedules)

UAT steps

  1. Pre-merge validation (done): YAML lint passes, no anchors/aliases present, only one helm ecosystem entry so directory overlap is impossible
  2. Post-merge: Confirm the .github/dependabot.yml check goes green on open PRs (e.g. feat(license-validation): add application example with custom field consumption and signature validation #138)
  3. Within a week: Confirm Dependabot opens correctly grouped PRs (replicated-sdk vs all-other-deps)

🤖 Generated with Claude Code

Dependabot requires unique (package-ecosystem, directory) pairs. The
previous config had two "helm" entries with identical directory lists,
which caused: "Update configs must have a unique combination of
'package-ecosystem', 'directory', and 'target-branch'."

Consolidate into a single entry with two groups:
- replicated-sdk: tracks the Replicated SDK dependency
- all-other-deps: tracks everything else (via exclude-patterns)

Both now run on a weekly cadence since Dependabot doesn't support
per-group schedules.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@kriscoleman kriscoleman requested a review from adamancini March 31, 2026 18:39
@kriscoleman
Copy link
Copy Markdown
Member Author

I'm tempted to update the dependabot trigger to trigger on this WIP branch for a full test... but at least its' passing yaml lint now

@adamancini adamancini merged commit 66af25f into main Mar 31, 2026
@adamancini adamancini deleted the kriscoleman/fix-dependabot-dirs branch March 31, 2026 18:53
@adamancini
Copy link
Copy Markdown
Member

adamancini commented Mar 31, 2026

@kriscoleman if you include a workflow_dispatch trigger, I think you could manually trigger the workflow to run against the test branch, and it should pick up your changes; as long as the workflow already exists on main, iirc.

@kriscoleman
Copy link
Copy Markdown
Member Author

@kriscoleman if you include a workflow_dispatch trigger, I think you could manually trigger the workflow to run against the test branch, and it should pick up your changes; as long as the workflow already exists on main, iirc.

that's a good idea! 💡

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.

2 participants