-
Notifications
You must be signed in to change notification settings - Fork 138
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
[main] Update community files #1995
[main] Update community files #1995
Conversation
Codecov ReportAll modified lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1995 +/- ##
==========================================
- Coverage 61.76% 60.00% -1.77%
==========================================
Files 106 106
Lines 13693 13693
==========================================
- Hits 8458 8217 -241
- Misses 4390 4633 +243
+ Partials 845 843 -2
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
/verify-owners |
e1e7684
to
df14e2b
Compare
/test |
@Cali0707: The
Use In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/test all |
df14e2b
to
aed2062
Compare
@Cali0707 hi gentle ping. I am trying to verify step 1 here: knative/release#247 for release-1.12. |
/test all |
@skonto I think any invalid owner files should be fixed now! |
Yeah btw PRs on other repo were merged 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.
/lgtm
/approve |
aed2062
to
4f94dc5
Compare
/lgtm |
Signed-off-by: Knative Automation <[email protected]>
4f94dc5
to
415fecd
Compare
@Cali0707 could you manually merge this one so we can move on? We will shortly have another PR with community files updates for 1.12, so we need to speed up this a bit. |
@skonto I don't have permissions to manually merge this |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: Cali0707, knative-automation, matejvasek, skonto The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/override "On Cluster RT Test (1.20.2, ubuntu-latest, s2i)" |
@matejvasek: matejvasek unauthorized: /override is restricted to Repo administrators. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/override "On Cluster RT Test (1.20.2, ubuntu-latest, s2i)" |
@lance: Overrode contexts on behalf of lance: On Cluster RT Test (1.20.2, ubuntu-latest, s2i) In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/override "On Cluster RT Test (1.20.2, ubuntu-latest, pack)" |
@lance: Overrode contexts on behalf of lance: On Cluster RT Test (1.20.2, ubuntu-latest, pack) In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Cron -knative-prow-robot
Produced by: knative-extensions/knobots/actions/update-community
Details: