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

fix: Update to latest sdk version #12

Merged
merged 1 commit into from
Sep 8, 2018

Conversation

rhuss
Copy link
Contributor

@rhuss rhuss commented May 19, 2018

  • Packages of SDK seems to have changed lately, so the template sample
    has been updated
  • Some minor documentation update

- Packages of SDK seems to have changed lately, so the template sample
  has been updated
- Some minor documentation update
@rhuss
Copy link
Contributor Author

rhuss commented May 19, 2018

Please note, that the README mentions that a "memcached" namespace should have been created, but this is not the case for me. The operator is running simply in the current (default) namespace. Not sure whether I did something wrong or whether the documentation is wrong at that point.

Otherwise everything went smoothely, thanks for this nice example.

@poussa
Copy link

poussa commented Aug 7, 2018

Why this PR has not been merged since it fixes an obvious issue?

@hasbro17
Copy link
Collaborator

hasbro17 commented Sep 8, 2018

Sorry for letting this sit for so long. We haven't been actively monitoring PRs and issues on this guide. We'll need to rethink about how to better keep maintain this guide and keep it in sync with the sdk repo docs.
LGTM.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants