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

fixing formatting in 1.25.0 doc and updating install prereq #6253

Merged
merged 1 commit into from
Jan 20, 2023

Conversation

acornett21
Copy link
Contributor

@acornett21 acornett21 commented Jan 19, 2023

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>
Copy link
Contributor

@everettraven everettraven left a comment

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
Copy link

gkurz commented Apr 26, 2023

@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