-
Notifications
You must be signed in to change notification settings - Fork 14.1k
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 guidance for reviewers about squashing commits #36364
Conversation
✅ Pull request preview available for checkingBuilt without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify site settings. |
124aaa2
to
47cd2f9
Compare
@mengjiao-liu @sftim rough draft. |
443404d
to
ef2aba1
Compare
The Kubernetes project currently lacks enough contributors to adequately respond to all issues and PRs. This bot triages issues and PRs according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /lifecycle stale |
The Kubernetes project currently lacks enough active contributors to adequately respond to all issues and PRs. This bot triages issues and PRs according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /lifecycle rotten |
/remove-lifecycle rotten |
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.
Few nits.
Hey @rolfedh . Are you actively working on the changes in this pull request? |
Hi @rolfedh! I'm providing another friendly ping on this PR. I understand you want to leave it open to review, so this is just a reminder as per PR Wrangling efforts. Thanks for your work so far! 👏 |
9c69314
to
e3bdb40
Compare
Hi @divya-mohan0209. Thanks for your comments. I've updated the PR and would appreciate your approval/merge. Rolfe |
@rolfedh I'd like to check that this should no longer be considered a work in progress? If so, I'd retitle the PR and remove the label. Also, you'll want to push an empty commit to retrigger the checks that are failing. /remove do-not-merge/work-in-progress |
e3bdb40
to
82ad2fc
Compare
Thanks for the guidance, @natalisucks. I think this is ready for merge. |
Thanks @rolfedh! 🤘 |
LGTM label has been added. Git tree hash: 72325348bc074c8f0b8bc1bc6f26e021432a5025
|
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.
I have a single suggestion and I'd be happy to approve this PR, once that is committed.
82ad2fc
to
cb1a807
Compare
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
LGTM label has been added. Git tree hash: a28e91601c87d1e2fae62c7a1c3735faab7460b1
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: divya-mohan0209 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 #36294