Skip to content

Conversation

@zacharysarah
Copy link
Contributor

@zacharysarah zacharysarah commented Jul 7, 2017

🛑 DO NOT MERGE 🛑

This PR's goal is to improve the Kubernetes contribution guidelines.

This megabranch consolidates multiple related PRs into a single topic branch.

PR Filename Description Contributing ⚡ Done
TBD README.md
  • Clarify the contribution process, consider sharing content with main Contributing page.
  • Provide a clear entry point for new contributors by linking to repo issues labeled P3 and Actionable.
TBD create-pull-request.md Creating a Documentation Pull Request:
  • Change title to Create a Pull Request
  • Change headings to:
    • Before you begin
    • Fork the Kubernetes documentation repository
    • Make your changes
    • Submit a pull request
    • Work with reviewers
  • Forking: make local changes in a topic branch on their fork, not master
  • Consolidate branch comparison sections

This change is Reviewable

@zacharysarah zacharysarah added the do-not-merge DEPRECATED. Indicates that a PR should not merge. Label can only be manually applied/removed. label Jul 7, 2017
@zacharysarah zacharysarah self-assigned this Jul 7, 2017
@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Jul 7, 2017
@zacharysarah zacharysarah changed the title [Work in progress] Updated contribution guidelines [Work in progress] Updating the contribution guidelines Jul 8, 2017
@steveperry-53
Copy link
Contributor

@zacharysarah, Don't move editdocs.md until you discussed it with the team. I seem to remember there's a reason that it is in its current location.

@chenopis
Copy link
Contributor

chenopis commented Aug 1, 2017

@zacharysarah @steveperry-53 So can we close this PR then?

@zacharysarah
Copy link
Contributor Author

@chenopis Yes, let's close it. We may come back to this, but it'll be a ways down the road if we do.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. do-not-merge DEPRECATED. Indicates that a PR should not merge. Label can only be manually applied/removed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants