Skip to content

Conversation

fabianvf
Copy link
Member

Description of the change:
make generate is now run before make prerelease, and the documentation is updated to match.

Motivation for the change:
This is a necessary step in the release process.

Checklist

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

@fabianvf fabianvf force-pushed the generate-prerelease branch from 99f603d to 0ed43e0 Compare May 27, 2021 17:51
Signed-off-by: Fabian von Feilitzsch <fabian@fabianism.us>
@fabianvf fabianvf force-pushed the generate-prerelease branch from 0ed43e0 to ac911e7 Compare May 27, 2021 18:01
@fabianvf fabianvf requested a review from estroz May 27, 2021 18:57
Copy link
Member

@estroz estroz left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label May 27, 2021
Copy link
Member

@jmrodri jmrodri left a comment

Choose a reason for hiding this comment

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

Looks just fine except for the typo.


```sh
git add --all
git add Makefile changelog website testdata
Copy link
Member

Choose a reason for hiding this comment

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

+1


```sh
git add --all
git add Makefile changelog website testdata
Copy link
Member

Choose a reason for hiding this comment

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

+1

@jmrodri
Copy link
Member

jmrodri commented Sep 3, 2021

Fixing typo will allow me to approve this.

@openshift-ci openshift-ci bot removed the lgtm Indicates that a PR is ready to be merged. label Sep 3, 2021
Co-authored-by: Jesus Rodriguez <jmrodri@gmail.com>
Signed-off-by: Fabian von Feilitzsch <fabian@fabianism.us>
@fabianvf fabianvf force-pushed the generate-prerelease branch from 57e9e7a to 86a008a Compare November 12, 2021 17:10
Copy link
Member

@jmrodri jmrodri left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Nov 12, 2021
@jmrodri jmrodri merged commit 311cd03 into operator-framework:master Nov 12, 2021
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