-
Notifications
You must be signed in to change notification settings - Fork 1.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove bypass for dependabot on dependency-review
check
#35066
Conversation
The bypass `if` for dependabot PRs added in #19745 is failing `dependency-review` due to this workflow effectively just proxying to another workflow in `shared-workflows`. This is causing the workflow to not run at all, and since it's required, it's causing dependabot PRs to not pass status checks. Remove the bypass for now to make checks pass again.
The PR changelog entry failed validation: Changelog entry not found in the PR body. Please add a "no-changelog" label to the PR, or changelog lines starting with |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, Reed!
@reedloden See the table below for backport results.
|
The bypass
if
for dependabot PRs added in #19745 is failingdependency-review
due to this workflow effectively just proxying to another workflow inshared-workflows
. This is causing the workflow to not run at all, and since it's required, it's causing dependabot PRs to not pass status checks. Remove the bypass for now to make checks pass again.