Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add new examples #515

Merged
merged 1 commit into from
Jun 11, 2018
Merged

Add new examples #515

merged 1 commit into from
Jun 11, 2018

Conversation

cdrage
Copy link
Member

@cdrage cdrage commented Jun 6, 2018

Adds a list of different examples (using each language that OpenShift
catalog supports) as well as updates the create.go command to list
each and every one of them!

@cdrage cdrage requested a review from kadel June 6, 2018 16:41
cmd/create.go Outdated
# Create a Python component
odo create python
# List of ready-to-use examples
odo create httpd --git https://github.com/openshift/httpd-ex.git
Copy link
Member

Choose a reason for hiding this comment

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

Does it really make sense to list all the possibilities here?
It looks really polluted. And to be honest is there any benefit of it?

I like the docs/examples.md document. But I'm not sure about listing everything here.

Examples here should be there just for users to help them understand the command variations and structure

Copy link
Member Author

Choose a reason for hiding this comment

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

We could include maybe one and a link to more examples? What do you think

Copy link
Contributor

Choose a reason for hiding this comment

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

something like, For more examples....

Copy link
Member

Choose a reason for hiding this comment

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

We could include maybe one and a link to more examples? What do you think

That would be much better.

@cdrage cdrage force-pushed the add-more-examples branch from 1f96fd1 to e795f86 Compare June 8, 2018 15:48
@surajnarwade
Copy link
Contributor

LGTM :)

cmd/create.go Outdated
# Create a Python component
odo create python
# List of ready-to-use examples
# for more examples, visit: https://github.com/redhat-developer/odo/blob/master/docs/examples.md
Copy link
Member

Choose a reason for hiding this comment

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

wrong indentation

cmd/create.go Outdated
odo create python
# List of ready-to-use examples
# for more examples, visit: https://github.com/redhat-developer/odo/blob/master/docs/examples.md
odo create php --git https://github.com/openshift/cakephp-ex.git
Copy link
Member

Choose a reason for hiding this comment

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

Would it make sense to remove this completely and just keep the note above?

Copy link
Member Author

@cdrage cdrage Jun 11, 2018

Choose a reason for hiding this comment

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

IMO adding something that is not NodeJS would help with our diversification of languages (showcase that all languages are first-class citizens)

Adds a list of different examples (using each language that OpenShift
catalog supports) as well as updates the `create.go` command to list
each and every one of them!
@cdrage cdrage force-pushed the add-more-examples branch from e795f86 to b77f742 Compare June 11, 2018 12:48
@cdrage
Copy link
Member Author

cdrage commented Jun 11, 2018

@kadel Updated!

@kadel kadel merged commit 5b998da into redhat-developer:master Jun 11, 2018
@cdrage cdrage deleted the add-more-examples branch July 3, 2018 16:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants