-
Notifications
You must be signed in to change notification settings - Fork 575
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
updated broken link #6786
updated broken link #6786
Conversation
Welcome @keshavcodex! It looks like this is your first PR to knative/eventing 🎉 |
Hi @keshavcodex. Thanks for your PR. I'm waiting for a knative member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. 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. |
SinkBinding runtime contract link is broken and i don't where this file has moved, can someone help me? |
in page ./docs/delivery/README.md these Source Delivery |
I would use a single link pointing to https://github.com/knative/specs/blob/main/specs/eventing/data-plane.md#event-delivery |
@keshavcodex: you cannot LGTM your own PR. 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. |
/ok-to-test |
Codecov ReportPatch and project coverage have no change.
Additional details and impacted files@@ Coverage Diff @@
## main #6786 +/- ##
=======================================
Coverage 80.45% 80.45%
=======================================
Files 236 236
Lines 12213 12213
=======================================
Hits 9826 9826
Misses 1896 1896
Partials 491 491 Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report in Codecov by Sentry. |
@keshavcodex As we talked in the WG meeting, we use Prow to manage PRs in Knative. One example where there's a failure with Prow jobs is here: knative-extensions/eventing-kafka-broker#2997 (comment) This is just an example. You can see All these being said, the failures in this PR is not failing Prow jobs. There are some Github actions that fail. These Github actions are used alongside with Prow. Current |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: aliok, keshavcodex 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 |
Fixes knative#5867 <!-- Please include the 'why' behind your changes if no issue exists --> I updated `NewConfigFromMap` broken link in [configmap.go](https://github.com/knative/pkg/blob/main/test/mako/config/configmap.go) file
Fixes knative#5867 <!-- Please include the 'why' behind your changes if no issue exists --> I updated `NewConfigFromMap` broken link in [configmap.go](https://github.com/knative/pkg/blob/main/test/mako/config/configmap.go) file
Fixes #5867
I updated
NewConfigFromMap
broken link in configmap.go file