-
Notifications
You must be signed in to change notification settings - Fork 504
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
🌱 Revert "perf.: run integration tests only on approved PRs (#2609)" #2612
Conversation
Since integration tests are required and currently broken, they will need to be marked as a not required check to get this merged in. |
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #2612 +/- ##
=======================================
Coverage 40.40% 40.40%
=======================================
Files 122 122
Lines 9908 9908
=======================================
Hits 4003 4003
Misses 5624 5624
Partials 281 281 |
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! I will mark the tests are not required for now.
This reverts commit a29182d. Signed-off-by: Spencer Schrock <sschrock@google.com>
2033887
to
aad61db
Compare
What kind of change does this PR introduce?
Bug fix for infra
What is the current behavior?
Reverts #2609
pull_request_review
doesn't have access to secrets, so the e2e tests are failinghttps://github.com/ossf/scorecard/actions/runs/3989324836
What is the new behavior (if this is a feature change)?**
should work as before
Which issue(s) this PR fixes
Special notes for your reviewer
Does this PR introduce a user-facing change?
For user-facing changes, please add a concise, human-readable release note to
the
release-note
(In particular, describe what changes users might need to make in their
application as a result of this pull request.)