Skip to content
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

Docs: Rename "Development" section to "Contributing" and add instructions to update the base AMI version of Ubuntu #10455

Merged
merged 4 commits into from
Dec 23, 2020

Conversation

moshevayner
Copy link
Member

Fixes #9705
/cc: @rifelpet @hakman

I've added a new section to the top panel named Contributing and moved the suggested topics from this comment under that section. It also made sense to me to move the main page of contributing from under Welcome to be the landing page of Contributing, hope you guys agree with this.
And lastly, I added a new doc describing the process of updating the base AMI versions of Ubuntu.

@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Dec 17, 2020
@moshevayner
Copy link
Member Author

/cc @rifelpet @hakman

Docs deploy preview is available here

@moshevayner
Copy link
Member Author

/retest

@olemarkus
Copy link
Member

I like the overall change. But I wonder if the development section can be a subsection of contributing.

@moshevayner
Copy link
Member Author

I like the overall change. But I wonder if the development section can be a subsection of contributing.

Yeah, that makes sense to me!

@k8s-ci-robot k8s-ci-robot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Dec 18, 2020
@moshevayner
Copy link
Member Author

/retest

Co-authored-by: Josh Branham <josh.php@gmail.com>
@moshevayner
Copy link
Member Author

@hakman @rifelpet hey guys! Just wanted to see if y'all had a chance to take a look at that yet?

mkdocs.yml Outdated
- Ports: "development/ports.md"
- Cluster Addons & Manager : "development/addons.md"
- Contributing:
- Getting Involved and Contributing: "contributing/contributing.md"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This file is a bit oddly named, with the URL being https://deploy-preview-10455--kubernetes-kops.netlify.app/contributing/contributing/ If we rename it to contributing/index.md does that shorten the URL to just https://deploy-preview-10455--kubernetes-kops.netlify.app/contributing?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yep, that worked! @rifelpet
image

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Comment on lines +135 to +138
- New Kubernetes Version: "contributing/new_kubernetes_version.md"
- Our Release Process: "contributing/release.md"
- Releasing With Homebrew: "contributing/homebrew.md"
- Updating The Default Base AMI: "contributing/update_ami_versions.md"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I do like that we have a Contributing section and a Development subsection and that it makes sense that Development is a subset of Contributing. I'm not sure it makes sense to have the entire Development section along side this handful of pages though, maybe we can put them under their own Subsection? I'm trying to think of a good name for it though... maybe some combination of these words:

common, repeat, periodic, release, processes, instructions, steps

but that would leave us with:

Contributing
- Some New Section
- Development

which might be awkward on the left side of the webpage to only have two menu items within Contributing. I don't have any good ideas here :(

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe we should just rename Development to Contributing :).

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe we should just rename Development to Contributing :).

Yeah I like that idea. I'll give that a try and see what it looks like

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@hakman
Copy link
Member

hakman commented Dec 23, 2020

Thanks @MoShitrit. It's ok from my point of view. Leaving final approval to @rifelpet or @olemarkus.
/approve

@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: hakman, MoShitrit

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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Dec 23, 2020
@moshevayner moshevayner changed the title Add dedicated "Contributing" section to the docs site Docs: Rename "Development" section to "Contributing" and add instructions to update the base AMI version of Ubuntu Dec 23, 2020
@rifelpet
Copy link
Member

yea I like how it is setup now, thanks for doing this!
/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Dec 23, 2020
@k8s-ci-robot k8s-ci-robot merged commit aff0ae2 into kubernetes:master Dec 23, 2020
@k8s-ci-robot k8s-ci-robot added this to the v1.20 milestone Dec 23, 2020
@moshevayner moshevayner deleted the docs/contributing branch December 23, 2020 17:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. area/documentation cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. lgtm "Looks good to me", indicates that a PR is ready to be merged. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Decide on a channels update policy for AMIs
6 participants