Skip to content

fixing formatting in 1.25.0 doc and updating install prereq#6253

Merged
everettraven merged 1 commit into
operator-framework:masterfrom
acornett21:fix_formatting
Jan 20, 2023
Merged

fixing formatting in 1.25.0 doc and updating install prereq#6253
everettraven merged 1 commit into
operator-framework:masterfrom
acornett21:fix_formatting

Conversation

@acornett21

@acornett21 acornett21 commented Jan 19, 2023

Copy link
Copy Markdown
Contributor

Description of the change:
Fixing some formatting issues in the 1.25.0 upgrade code and change the instillation prereq to be go 1.19

Motivation for the change:
When following the upgrade from 1.22 to 1.26 I noticed some formating issues in the 1.25.0 doc that caused them to not copy properly, and also missing the need to change from go 1.18 to go 1.19.

Checklist

If the pull request includes user-facing changes, extra documentation is required:

Signed-off-by: Adam D. Cornett adc@redhat.com

…1.19

Signed-off-by: Adam D. Cornett <adc@redhat.com>
@openshift-ci openshift-ci Bot requested review from grokspawn and jberkhahn January 19, 2023 14:24

@everettraven everettraven left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@acornett21 Thanks for the contribution! These changes look good to me!

/lgtm

@openshift-ci openshift-ci Bot added the lgtm Indicates that a PR is ready to be merged. label Jan 19, 2023
@everettraven everettraven merged commit a95c9f2 into operator-framework:master Jan 20, 2023
@acornett21 acornett21 deleted the fix_formatting branch January 20, 2023 23:38
@gkurz

gkurz commented Apr 26, 2023

Copy link
Copy Markdown

@everettraven I would expect to see this change live in https://v1-26-x.sdk.operatorframework.io/docs/installation/ but it still mentions go 1.18... am I missing something ?

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

Labels

lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants