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

Make all docs have a warning if unversioned #11124

Merged

Conversation

thockin
Copy link
Member

@thockin thockin commented Jul 12, 2015

Use mungedocs to insert a header on every .md file that screams "This is an unversioned doc!". Examples are not covered yet, since we don't run mungedocs on examples, but I will add that.

Suggest reviewing commit-by-commit. They might make a bit more sense in a different order but it was not worth the effort to rejigger.

@lavalamp - no more running mungedocs on TMP_DOCROOT - yay!

@brendandburns @zmerlynn

@k8s-bot
Copy link

k8s-bot commented Jul 12, 2015

GCE e2e build/test passed for commit 112ae11.

@thockin thockin force-pushed the docs-munge-branch-warnings_filter_munges branch from 112ae11 to 858f7b5 Compare July 13, 2015 17:23
@thockin
Copy link
Member Author

thockin commented Jul 13, 2015

Given 3 people working on munges, I'd like to call dibs on not rebasing, since I am oncall today.

@zmerlynn
Copy link
Member

Looks totally plausible. I didn't stare too long yet, though.

@k8s-bot
Copy link

k8s-bot commented Jul 13, 2015

GCE e2e build/test passed for commit 858f7b588416a8e17aded720647000429de2af99.

@bgrant0607
Copy link
Member

cc @mikedanese @erictune

@erictune
Copy link
Member

LGTM

@erictune erictune added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Jul 13, 2015
@erictune
Copy link
Member

merge it! :shipit:

thockin added 6 commits July 13, 2015 11:23
Mungedocs will keep this updated in head, and it will get stricken from
releases.
All munges now start with `<!-- BEGIN MUNGE:` and end with `<!-- END MUNGE:`.
This lets me (in a followup) filter them better to normalize contents during
verification of generated docs.
This allows us to apply content-mutating munges to generated files and nothave
the verifier flag them as needing regeneration later.
@mikedanese
Copy link
Member

need rebase

@thockin thockin force-pushed the docs-munge-branch-warnings_filter_munges branch from 858f7b5 to fffc64e Compare July 13, 2015 18:28
thockin added a commit that referenced this pull request Jul 13, 2015
…ter_munges

Make all docs have a warning if unversioned
@thockin thockin merged commit 64e3047 into kubernetes:master Jul 13, 2015
@thockin
Copy link
Member Author

thockin commented Jul 13, 2015

merged

@k8s-bot
Copy link

k8s-bot commented Jul 13, 2015

GCE e2e build/test passed for commit fffc64e.

@nikhiljindal
Copy link
Contributor

This PR contains changes to build/mark-new-version which havent been cherrypicked to 1.0.
Is that intended?

@thockin
Copy link
Member Author

thockin commented Jul 20, 2015

not intended, can you handle it?

On Mon, Jul 20, 2015 at 1:40 PM, Nikhil Jindal notifications@github.com
wrote:

This PR contains changes to build/mark-new-version which havent been
cherrypicked to 1.0.
Is that intended?


Reply to this email directly or view it on GitHub
#11124 (comment)
.

@nikhiljindal
Copy link
Contributor

Yes. Am handling it

@thockin thockin deleted the docs-munge-branch-warnings_filter_munges branch July 27, 2015 15:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm "Looks good to me", indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants