Skip to content
This repository was archived by the owner on Sep 21, 2020. It is now read-only.

Conversation

@estroz
Copy link
Member

@estroz estroz commented Nov 2, 2018

This PR brings the getting-started user guide in sync with operator-sdk's user guide.

@ecordell can you make sure the OLM-related portion is up to date (perhaps in a separate PR)? Thanks!

Closes #13 closes #15 closes #16 closes #18 closes #25

@openshift-ci-robot openshift-ci-robot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Nov 2, 2018
@hasbro17
Copy link
Collaborator

hasbro17 commented Nov 5, 2018

/cc @adellape

Copy link
Collaborator

@AlexNPavel AlexNPavel left a comment

Choose a reason for hiding this comment

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

A couple of minor issues, otherwise lgtm

AlexNPavel and others added 3 commits November 5, 2018 15:17
Co-Authored-By: estroz <ericstroczynski@gmail.com>
Co-Authored-By: estroz <ericstroczynski@gmail.com>
Copy link
Collaborator

@hasbro17 hasbro17 left a comment

Choose a reason for hiding this comment

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

LGTM

@estroz estroz merged commit ccefb7b into operator-framework:master Nov 6, 2018
@estroz estroz deleted the update-guide branch November 6, 2018 20:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

4 participants