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

Update README #51

Merged
merged 2 commits into from
Sep 11, 2019
Merged

Update README #51

merged 2 commits into from
Sep 11, 2019

Conversation

akoserwal
Copy link
Contributor

Updated create new project steps based on https://github.com/operator-framework/operator-sdk/blob/master/README.md

INFO[0000] Creating new Go operator 'memcached-operator'. 
Error: dependency manager "modules" requires working directory to be in $GOPATH/src and GO111MODULE=on, or outside of $GOPATH/src and GO111MODULE="on", "auto", or unset. More info: https://github.com/operator-framework/operator-sdk/blob/master/doc/user-guide.md#go-modules

Updated create new project steps based on https://github.com/operator-framework/operator-sdk/blob/master/README.md,
```
INFO[0000] Creating new Go operator 'memcached-operator'. 
Error: dependency manager "modules" requires working directory to be in $GOPATH/src and GO111MODULE=on, or outside of $GOPATH/src and GO111MODULE="on", "auto", or unset. More info: https://github.com/operator-framework/operator-sdk/blob/master/doc/user-guide.md#go-modules
```
@openshift-ci-robot openshift-ci-robot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Sep 11, 2019
Removing `--repo github.com/example-inc/memcached-operator`.
@camilamacedo86
Copy link
Contributor

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Sep 11, 2019
Copy link
Contributor

@camilamacedo86 camilamacedo86 left a comment

Choose a reason for hiding this comment

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

great 👍

@camilamacedo86
Copy link
Contributor

Hi @estroz and @joelanford,

Have you any objection? Could we move forward here?

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

@estroz estroz merged commit 2be3e81 into operator-framework:master Sep 11, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
lgtm Indicates that a PR is ready to be merged. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants