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 approvers list for dev-console/integration-tests folder #5739
Updated approvers list for dev-console/integration-tests folder #5739
Conversation
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
/lgtm cancel @makambalaji Can you squash your commits? I just noticed this now after I approved. Once you squash anyone can give you a LGTM if I am busy. |
@makambalaji I suspect you're going to want an owners file in the |
Agreed, but it's not part of dev console right, So I doubt, whether I can ask approval for that folder or not |
The dev-console team maintains the |
Ok, then I will add another commit to the same PR, along with squashing the commits |
update the owners list for knative-pluign/integration-tests
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
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: andrewballantyne, makambalaji 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 |
/retest Somehow test failures when OWNERS files are added? 🤔 Gotta be a fluke |
/retest Heh, different test failures now. |
/test e2e-gcp-console No logs? Hmmm..... trying again |
As per the today's discussion in UI sync meeting, updated OWNERS doc in integration-tests folder by including "makambalaji" in approvers list