Skip to content
This repository has been archived by the owner on Jan 3, 2023. It is now read-only.

consolidate demo/README.md and examples/zone-printer/README.md #161

Open
ahmetb opened this issue Mar 29, 2018 · 2 comments
Open

consolidate demo/README.md and examples/zone-printer/README.md #161

ahmetb opened this issue Mar 29, 2018 · 2 comments

Comments

@ahmetb
Copy link
Contributor

ahmetb commented Mar 29, 2018

I think you should move zone-printer/README.md to demo/.

We can easily change zone-printer with a different app that no longer uses nginx+configmap and the demo/sample/example isn't much about the app itself, but rather instructions.

@ahmetb
Copy link
Contributor Author

ahmetb commented Mar 29, 2018

I understand demo/ is for bash prompt automation. Consider moving it out of project root. It's not an important component and users will not use it.

@nikhiljindal
Copy link
Contributor

Yes demo/ has scripts that people can use to demo kubemci. We could move it out of the repo, it is there for discoverability.

I dont see this as an important issue.

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

No branches or pull requests

2 participants