Skip to content

chore(ci): add dependabot config for automated dependency updates - #21

Closed
jmeridth wants to merge 2 commits into
EpicGames:mainfrom
jmeridth:feat/add-dependabot-config
Closed

chore(ci): add dependabot config for automated dependency updates#21
jmeridth wants to merge 2 commits into
EpicGames:mainfrom
jmeridth:feat/add-dependabot-config

Conversation

@jmeridth

@jmeridth jmeridth commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

What/Why

Add Dependabot configuration to automate dependency update PRs across ecosystems in the repo: uv (Python), github-actions, and docker (lore-server, lore-revision). Cargo was excluded per maintainer feedback.

Proof it works

YAML validates cleanly. Dependabot will pick up the config on merge and begin scanning on its weekly schedule.

Risk + AI role

Low -- config-only addition, no code changes. AI-assisted (Claude Opus 4.6).

Review focus

  • Confirm the grouped minor+patch strategy is appropriate and that no ecosystems were missed.
  • The labels (python, github_actions, docker, dependencies) must exist in the repo or be created before merge, otherwise Dependabot PRs will fail to apply them.

What/Why

Enable Dependabot to automatically open PRs for outdated dependencies
across all ecosystems in the repo: cargo, uv, github-actions, and
docker (lore-server, lore-revision).

Proof it works

YAML validates cleanly. Dependabot will pick up the config on merge
and begin scanning on its weekly schedule.

Risk + AI role

Low -- config-only addition, no code changes. AI-generated
(Claude Opus 4.6, claude-opus-4-6), human-reviewed.

Review focus

Confirm the grouped minor+patch strategy is appropriate and that no
ecosystems were missed. The labels (rust, python, github_actions,
docker, dependencies) must exist in the repo or be created before
merge, otherwise Dependabot PRs will fail to apply them.

Signed-off-by: jmeridth <jmeridth@gmail.com>
@jmeridth
jmeridth marked this pull request as ready for review June 18, 2026 03:59
@ragnarula

Copy link
Copy Markdown
Collaborator

Hi thanks for the contribution @jmeridth.

We check rust dependencies in our internal CI with cargo deny already. We'll work to try to make that more visible. Is there some overlap with this?

@jmeridth

jmeridth commented Jun 18, 2026

Copy link
Copy Markdown
Contributor Author

Hi thanks for the contribution @jmeridth.

We check rust dependencies in our internal CI with cargo deny already. We'll work to try to make that more visible. Is there some overlap with this?

UPDATE: DONE

We could remove the cargo ecosystem from the file and leave a comment for that. Up to y'all. Are the other bits valid? (GitHub Actions, uv (Python), Docker)

## What/Why

Remove the cargo package ecosystem from the dependabot configuration per
PR review feedback (EpicGames#21).

## Proof it works

YAML-only change; validated structure manually. Remaining ecosystems
(uv, github-actions, docker) are unchanged.

## Risk + AI role

Low -- config removal only. AI-assisted (Claude Opus 4.6) for the edit.

## Review focus

Confirm cargo removal aligns with upstream maintainer intent.

Signed-off-by: jmeridth <jmeridth@gmail.com>

@ragnarula ragnarula left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I'm happy to give this a go, we've been watching our rust dependencies but not so much other things so seems like a valuable tool.

If you wouldn't mind could you

  • Run this on your fork to prove it runs cleanly and also any updates it picks up don't immediately bring in breaking changes
  • Rebase/merge main to pick up our latest PR checks

If thats all good I'm happy to take it once our merge flow comes online.

Left a comment about the mysterious docker file, if that gets removed before this merges lets remove that section too.

Comment thread .github/dependabot.yaml
- "minor"
- "patch"
- package-ecosystem: "docker"
directory: "/lore-revision"

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Seeing this actually brought to my attention this docker file exists. I'm not sure why its here or whats using it, if I can't find a consumer I'll probably remove it so probably should remove this too.

jmeridth added a commit to jmeridth/lore that referenced this pull request Jun 19, 2026
## What/Why

Remove the cargo package ecosystem from the dependabot configuration per
PR review feedback (EpicGames#21).

## Proof it works

YAML-only change; validated structure manually. Remaining ecosystems
(uv, github-actions, docker) are unchanged.

## Risk + AI role

Low -- config removal only. AI-assisted (Claude Opus 4.6) for the edit.

## Review focus

Confirm cargo removal aligns with upstream maintainer intent.

Signed-off-by: jmeridth <jmeridth@gmail.com>
@jmeridth

jmeridth commented Jun 19, 2026

Copy link
Copy Markdown
Contributor Author

I'm happy to give this a go, we've been watching our rust dependencies but not so much other things so seems like a valuable tool.

If you wouldn't mind could you

  • Run this on your fork to prove it runs cleanly and also any updates it picks up don't immediately bring in breaking changes
  • Rebase/merge main to pick up our latest PR checks

If thats all good I'm happy to take it once our merge flow comes online.

Left a comment about the mysterious docker file, if that gets removed before this merges lets remove that section too.

@ragnarula

Merged to my fork's main branch

Enabled dependabot and ran the checks

Checks running

Screenshot 2026-06-19 at 08 52 55

Resulting PRs (count: 7)

https://github.com/jmeridth/lore/pulls

Screenshot 2026-06-19 at 08 54 26

The Actions updates would be solved by my other PR

@ajcarberry
ajcarberry self-requested a review July 14, 2026 14:07
@ragnarula ragnarula added the ready-to-import Approved by Epic staff for import into Lore label Jul 14, 2026
@epic-lore-bot

epic-lore-bot Bot commented Jul 14, 2026

Copy link
Copy Markdown

Imported as Lore CR-208.

@epic-lore-bot epic-lore-bot Bot added imported Imported into Lore for internal review and removed ready-to-import Approved by Epic staff for import into Lore labels Jul 14, 2026
@concernedrat concernedrat added ready-to-import Approved by Epic staff for import into Lore and removed imported Imported into Lore for internal review labels Jul 15, 2026
@epic-lore-bot epic-lore-bot Bot added imported Imported into Lore for internal review and removed ready-to-import Approved by Epic staff for import into Lore labels Jul 15, 2026
@epic-lore-bot

epic-lore-bot Bot commented Jul 15, 2026

Copy link
Copy Markdown

Imported as Lore CR-215.

epic-lore-bot Bot pushed a commit that referenced this pull request Jul 20, 2026
## What/Why

Add Dependabot configuration to automate dependency update PRs across ecosystems in the repo: uv (Python), github-actions, and docker (lore-server, lore-revision). Cargo was excluded per maintainer feedback.

## Proof it works

YAML validates cleanly. Dependabot will pick up the config on merge and begin scanning on its weekly schedule.

## Risk + AI role

Low -- config-only addition, no code changes. AI-assisted (Claude Opus 4.6).

## Review focus

- Confirm the grouped minor+patch strategy is appropriate and that no ecosystems were missed.
- The labels (python, github_actions, docker, dependencies) must exist in the repo or be created before merge, otherwise Dependabot PRs will fail to apply them.

```
Imported-PR: #21
Imported-From: 6b975e9
Imported-Base: a00b1c4
Imported-Merge: 4c01812
Imported-Author: jmeridth (jmeridth)
Signed-off-by: jmeridth <jmeridth@gmail.com>
GH-URL: #21
```

Lore-RevId: 320
Lore-Signature: 24ad217c0625277bef8fb990f0bd34ca0fb5f06a46ae57a91849783198e9f19b
@epic-lore-bot

epic-lore-bot Bot commented Jul 20, 2026

Copy link
Copy Markdown

Closed by mirrored commit c0525a6.

@epic-lore-bot epic-lore-bot Bot closed this Jul 20, 2026
@epic-lore-bot epic-lore-bot Bot added the merged Merged into Lore codebase label Jul 20, 2026
@jmeridth
jmeridth deleted the feat/add-dependabot-config branch July 20, 2026 12:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

imported Imported into Lore for internal review merged Merged into Lore codebase

Development

Successfully merging this pull request may close these issues.

3 participants