-
Notifications
You must be signed in to change notification settings - Fork 5.2k
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
Adding the section of contributor guide for sig-windows #3414
Conversation
/milestone v1.14 |
no longer need this file since the content is now in the KEP and the user-facing documentation
/assign castrojo |
@nikopen can you please add this to 1.14 milestone? |
Hi @michmike - thank you for this PR! I'm going to take a look at this tomorrow. For now, know that milestones are only enabled on kubernetes/kubernetes, so we won't need to follow release process here. |
Indeed milestones are only needed in the core repo. Structure wise, I'd suggest the contributor guide to be moved within the SIG-Windows directory. @guineveresaenger On the central 'contributors/guide/readme.md' there should be links to every single SIG's contributor guide so people can easily be directed to a more specific contributing resource. |
/hold |
that works for me if that's what the central guidance will be |
Wow thanks for this, agree with the rest, have |
i moved and re-arranged the files as requested. how does it look now? |
/hold cancel |
/lgtm |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: castrojo, michmike 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 |
Hi everyone! I have some different thoughts on this, and I literally just started my work day.
I'm completely open to making these SIG-specific guides more visible via @nikopen's comment, but the contributor guide should not cater to individual SIGs or technologies; it will result in too much clutter. |
@guineveresaenger |
i agree with @parispittman 100% on 1 and 2. if you are all good with it, i can create a new PR to reflect the new changes |
Hi friends! I'm just working on incorporating this feedback, a new PR will be up momentarily! Thank you for your patience, @michmike :) |
Here it is: #3419 |
👍 |
No description provided.