-
Notifications
You must be signed in to change notification settings - Fork 77
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
Add dependabot and auto hack scripts #830
Conversation
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: nak3 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 |
Codecov ReportPatch and project coverage have no change.
Additional details and impacted files@@ Coverage Diff @@
## main #830 +/- ##
=======================================
Coverage 82.84% 82.84%
=======================================
Files 45 45
Lines 1708 1708
=======================================
Hits 1415 1415
Misses 252 252
Partials 41 41 ☔ View full report in Codecov by Sentry. |
/hold |
@@ -0,0 +1,38 @@ | |||
name: dependabot - hack scripts |
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.
This should be added to either actions or .github repo so that we can just sync the file across repos. Not everyone uses dependabot but there are a few that do and would benefit from it.
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.
Okay, I was hesitated to adding the common dependabot because some repo has their own dependabot like https://github.com/knative-extensions/eventing-kafka-broker/blob/main/.github/workflows/build-dependabot-pr.yaml
But I will add it to the common one.
We already merged this dependabot workflow and it is my bad for not mentioning this then. However, we should generalize this and move it to |
As per title, this patch adds dependabot.
/cc @kvmware @dprotaso