Skip to content

rn-4-14-clean: Scan 4.14 Release Notes for typos, and so on#67005

Merged
skrthomas merged 1 commit intoopenshift:enterprise-4.14from
dfitzmau:rn-4-14-clean
Oct 27, 2023
Merged

rn-4-14-clean: Scan 4.14 Release Notes for typos, and so on#67005
skrthomas merged 1 commit intoopenshift:enterprise-4.14from
dfitzmau:rn-4-14-clean

Conversation

@dfitzmau
Copy link
Contributor

@dfitzmau dfitzmau commented Oct 27, 2023

[INTERNAL FIX FROM RELEASE NOTES TEAM]

Version(s):
4.14

Link to docs preview:
OCP 4.14 RN doc

QE review: No QE approval needed.

Additional information:
Pre-published 4.14 RN doc

@openshift-ci openshift-ci bot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Oct 27, 2023
@ocpdocs-previewbot
Copy link

ocpdocs-previewbot commented Oct 27, 2023

🤖 Updated build preview is available at:
https://67005--docspreview.netlify.app

Build log: https://circleci.com/gh/ocpdocs-previewbot/openshift-docs/30684

@openshift-ci openshift-ci bot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Oct 27, 2023
@dfitzmau dfitzmau force-pushed the rn-4-14-clean branch 6 times, most recently from 3937e93 to 7cd69e4 Compare October 27, 2023 11:28
@dfitzmau
Copy link
Contributor Author

/label peer-review-needed

@openshift-ci openshift-ci bot added the peer-review-needed Signifies that the peer review team needs to review this PR label Oct 27, 2023
@dfitzmau dfitzmau force-pushed the rn-4-14-clean branch 2 times, most recently from 465b5a2 to 7c7d16a Compare October 27, 2023 13:04
@ShaunaDiaz
Copy link
Contributor

/label peer-review-in-progress

@openshift-ci openshift-ci bot added the peer-review-in-progress Signifies that the peer review team is reviewing this PR label Oct 27, 2023
@ShaunaDiaz
Copy link
Contributor

/remove-label peer-review-needed

@openshift-ci openshift-ci bot removed the peer-review-needed Signifies that the peer review team needs to review this PR label Oct 27, 2023
Copy link
Contributor

@ShaunaDiaz ShaunaDiaz left a comment

Choose a reason for hiding this comment

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

Nits on nits, right? As always, your call on which edits to accept. These look great!

@@ -2901,11 +2925,11 @@ For any {product-title} release, always review the instructions on xref:../updat

//Update with relevant advisory information
[id="ocp-4-14-0-ga"]
=== RHSA-2023:1234 - {product-title} 4.14.0 image release, bug fix, and security update advisory
=== RHSA-2023:1234 - {product-title} {product-version}.0 image release, bug fix, and security update advisory
Copy link
Contributor

Choose a reason for hiding this comment

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

I think we have the ART Errata now? Can probably update these, too.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good catch, Shauna. Updated to match the advisory numbers on https://issues.redhat.com/browse/ART-7691.

@JoeAldinger . A further update is due for this section based on the completion of https://issues.redhat.com/browse/OSDOCS-8362 ?

@ShaunaDiaz
Copy link
Contributor

/label peer-review-done

@openshift-merge-robot openshift-merge-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Oct 27, 2023
@ShaunaDiaz
Copy link
Contributor

/remove-label peer-review-in-progress

@openshift-ci openshift-ci bot added peer-review-done Signifies that the peer review team has reviewed this PR and removed peer-review-in-progress Signifies that the peer review team is reviewing this PR labels Oct 27, 2023
@openshift-merge-robot openshift-merge-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Oct 27, 2023
@dfitzmau
Copy link
Contributor Author

/label merge-review-needed

@openshift-ci openshift-ci bot added the merge-review-needed Signifies that the merge review team needs to review this PR label Oct 27, 2023
@skrthomas skrthomas added merge-review-in-progress Signifies that the merge review team is reviewing this PR and removed merge-review-needed Signifies that the merge review team needs to review this PR labels Oct 27, 2023
@skrthomas
Copy link
Contributor

Very thorough good fixes here! 🔍 /lgtm!

@skrthomas skrthomas merged commit f7f87f1 into openshift:enterprise-4.14 Oct 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

merge-review-in-progress Signifies that the merge review team is reviewing this PR peer-review-done Signifies that the peer review team has reviewed this PR size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants