Skip to content

Remove Dependabot PR approval step#255

Merged
bourgeoa merged 1 commit intomainfrom
bourgeoa-npm-1
Apr 14, 2026
Merged

Remove Dependabot PR approval step#255
bourgeoa merged 1 commit intomainfrom
bourgeoa-npm-1

Conversation

@bourgeoa
Copy link
Copy Markdown
Contributor

Removed approval step for Dependabot PRs in CI workflow.

Removed approval step for Dependabot PRs in CI workflow.
Copilot AI review requested due to automatic review settings April 14, 2026 08:34
@bourgeoa bourgeoa enabled auto-merge April 14, 2026 08:34
@bourgeoa bourgeoa merged commit aae618b into main Apr 14, 2026
8 checks passed
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Removes the explicit “Approve Dependabot PR” step from the CI workflow’s Dependabot automation, leaving auto-merge enablement as the remaining Dependabot-specific action.

Changes:

  • Deleted the GitHub CLI step that auto-approves Dependabot-authored pull requests.
  • Kept the existing auto-merge enablement logic for semver patch/minor Dependabot updates.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
- name: Enable auto-merge for Dependabot PRs
if: steps.metadata.outputs.update-type == 'version-update:semver-patch' || steps.metadata.outputs.update-type == 'version-update:semver-minor'
run: gh pr merge --auto --merge "$PR_URL" # Use Github CLI to merge automatically the PR
GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
- name: Enable auto-merge for Dependabot PRs
if: steps.metadata.outputs.update-type == 'version-update:semver-patch' || steps.metadata.outputs.update-type == 'version-update:semver-minor'
run: gh pr merge --auto --merge "$PR_URL" # Use Github CLI to merge automatically the PR
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