-
Notifications
You must be signed in to change notification settings - Fork 8
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
Drop OpenShiftShort attribute #532
Conversation
The documentation style indicates we shouldn't be using OCP or OKD short forms of OpenShift, so I'm dropping that.
So here is a proposal for dropping OpenShiftShort. Alternatively, could we maybe set the value to |
How about setting the value to RHOCP? |
I think that defeats the purpose. My understanding is we're not supposed to abbreviate OpenShift at all. |
Ok I'll double check with writing team. I can do another commit to #531 that includes the update from this PR (532) if that suits |
We only used OpenShiftShort in 2 spots previously, so I would drop all use of it to maintain consistency in the guide. Since we never used it before, I would recommend it be removed everywhere, and not added in the new disconnected contents. |
These changes are incorporated into #531 |
Do doc-Service-Telemetry-Framework/modules/proc_overriding-a-managed-manifest.adoc and doc-Service-Telemetry-Framework/modules/proc_overriding-a-managed-manifest.adoc get published in the STF guide? |
Doesn't seem so. I have opened #533 to drop the one file that still exists. Doesn't get included anywhere. |
The documentation style indicates we shouldn't be using OCP or OKD short
forms of OpenShift, so I'm dropping that.