Skip to content

Add Dependabot auto-merge workflow - #108

Open
mosuem wants to merge 1 commit into
mainfrom
dependabot-auto-merge
Open

Add Dependabot auto-merge workflow#108
mosuem wants to merge 1 commit into
mainfrom
dependabot-auto-merge

Conversation

@mosuem

@mosuem mosuem commented Aug 20, 2026

Copy link
Copy Markdown
Collaborator

Follows the official GitHub Dependabot automation guidelines to automatically approve and enable auto-merge for Dependabot pull requests once all status checks pass.

@mosuem
mosuem requested a review from DanTup August 20, 2026 09:14
@mosuem
mosuem marked this pull request as ready for review August 20, 2026 09:14

@DanTup DanTup 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.

Some minor questions, but lgtm!

jobs:
dependabot:
runs-on: ubuntu-latest
if: github.event.pull_request.user.login == 'dependabot[bot]'

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.

In the example in the linked page, it also has a condition like && github.repository == 'owner/my_repo'. I'm not certain, but I think without it, this might also run on peoples forks - is that desirable? Should there be a similar filter here?

Comment on lines +13 to +17
- name: Dependabot metadata
id: metadata
uses: dependabot/fetch-metadata@d7267f607e9d3fb96fc2fbe83e0af444713e90b7 # v2.3.0
with:
github-token: "${{ secrets.GITHUB_TOKEN }}"

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.

Is this step doing anything? It looks like it would be used if you need to check the metadata, but its outputs are not used in further steps (it looks like one of GH's examples also does this, though some others do use the outputs).

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