Conversation
Agent-Logs-Url: https://github.com/USTC-KnowledgeComputingLab/ddss/sessions/0edc6775-89d5-4b9d-ba2d-aa49a24499ee Co-authored-by: hzhangxyz <11623447+hzhangxyz@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Adds a GitHub Actions workflow to automatically enable GitHub native auto-merge for Dependabot PRs, so those PRs merge automatically once all required checks pass.
Changes:
- Introduces
.github/workflows/dependabot.ymlto run onpull_requestevents. - Gates the job to Dependabot activity and enables auto-merge via
gh pr merge --auto --squash.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| jobs: | ||
| dependabot: | ||
| runs-on: ubuntu-latest | ||
| if: github.actor == 'dependabot[bot]' |
There was a problem hiding this comment.
The job condition uses github.actor == 'dependabot[bot]', which will stop enabling auto-merge if a non-Dependabot user triggers a pull_request event on a Dependabot PR (e.g., via “Update branch”/rebases that generate a new synchronize event). Consider gating on the PR author instead (e.g., github.event.pull_request.user.login) so the workflow consistently applies to Dependabot-authored PRs regardless of who triggered the latest event.
| if: github.actor == 'dependabot[bot]' | |
| if: github.event.pull_request.user.login == 'dependabot[bot]' |
| @@ -0,0 +1,19 @@ | |||
| name: dependabot | |||
There was a problem hiding this comment.
Workflow names in this repo appear to be Title Case (e.g., “Docs”, “Jest”, “Pytest”). To match that convention, consider renaming this workflow from dependabot to something like “Dependabot”.
| name: dependabot | |
| name: Dependabot |
Mirrors the functionality from USTC-KnowledgeComputingLab/ds#318: automatically merge Dependabot PRs once all required status checks pass.
Changes
.github/workflows/dependabot.yml: triggers onpull_request, gates ongithub.actor == 'dependabot[bot]', and enables GitHub's native auto-merge viagh pr merge --auto --squash— the actual merge only fires after all required checks passRequires
contents: writeandpull-requests: writepermissions. Style follows existing workflows in this repo (on:list notation, same job structure).