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
Reduces the boilerplate of our controller binaries. #3503
Conversation
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: mattmoor 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 |
efe4897
to
4a5fb76
Compare
Coverage dropped to below 50%, is that fixable? |
/hold Yeah, definitely not intended, but what I get for moving code out of ignored directories! :) |
4a5fb76
to
e234831
Compare
/hold Too many failures, need to dig deeper. |
e234831
to
45370a8
Compare
Forgot that I switched this to blocking when I moved to knative/pkg. This should be RFAL now. I also filed #3508 which was the somewhat unintuitive failure more I was seeing. /hold cancel |
The following is the coverage report on pkg/.
|
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
No description provided.