Skip to content

auto approve should process npm package uupdates#45

Merged
simongdavies merged 2 commits intohyperlight-dev:mainfrom
simongdavies:fix-auto-approve
Apr 9, 2026
Merged

auto approve should process npm package uupdates#45
simongdavies merged 2 commits intohyperlight-dev:mainfrom
simongdavies:fix-auto-approve

Conversation

@simongdavies
Copy link
Copy Markdown
Contributor

This pull request updates the scripts/auto-approve-dependabot.sh script to expand its support for auto-approving and merging Dependabot pull requests. The script now includes support for PRs that update npm dependencies, in addition to Rust dependencies, and allows changes to the relevant npm files.

Signed-off-by: Simon Davies <simongdavies@users.noreply.github.com>
@simongdavies simongdavies added the bug Something isn't working label Apr 9, 2026
Copilot AI review requested due to automatic review settings April 9, 2026 12:51
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

This pull request expands the Dependabot auto-approve/merge workflow by extending scripts/auto-approve-dependabot.sh to include Dependabot npm (dependabot/npm_and_yarn) PRs and to permit npm manifest/lockfile changes.

Changes:

  • Include Dependabot branches prefixed with dependabot/npm_and_yarn in the PR selection filter.
  • Allow auto-merge when modified files include package.json and package-lock.json (in addition to Cargo.toml/Cargo.lock).
  • Update script output messages to reflect the expanded allowed file set.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@simongdavies simongdavies enabled auto-merge (squash) April 9, 2026 13:30
@simongdavies simongdavies merged commit 5fccd4c into hyperlight-dev:main Apr 9, 2026
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants