Skip to content

Dependency Review triggers#50

Open
akhanhextech wants to merge 6 commits into
mainfrom
akhan/concurrency
Open

Dependency Review triggers#50
akhanhextech wants to merge 6 commits into
mainfrom
akhan/concurrency

Conversation

@akhanhextech
Copy link
Copy Markdown
Collaborator

Description

Occasionally the GH Action for Dependency Review stays stuck waiting for the workflow to kick off. That may be correlated to rebases (like changing the base to main), but unclear. Making the following changes as potential fixes.

  1. Adding "concurrency" config to cancel older runs. Just in case we're triggering several runs, causing a backlog.
  2. Adding "edited" event as trigger, to make sure we trigger on rebase.

Testing

Comment thread .github/workflows/dependency-review.yml Outdated
Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit a8b4a40. Configure here.

- labeled
- unlabeled
- reopened
- edited
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Unmentioned removal of labeled/unlabeled event triggers

Low Severity

The labeled and unlabeled pull request event triggers were removed, but the PR description only mentions adding the edited trigger and concurrency config — it doesn't mention removing any existing triggers. These triggers may have been used to allow re-running dependency review by toggling a label on a PR. This looks like an accidental removal during editing of the trigger list.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit a8b4a40. Configure here.

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.

1 participant