Skip to content

Conversation

jc-berger
Copy link
Contributor

@jc-berger jc-berger added the documentation Improvements or additions to documentation label Feb 3, 2022
@jc-berger jc-berger self-assigned this Feb 3, 2022
@github-actions
Copy link

github-actions bot commented Feb 3, 2022

Deploy PR Preview failed.

@jc-berger
Copy link
Contributor Author

@kim-tsao thanks so much for the review and help with the yaml examples :)

Please see the new commit, applied dev feedback and added ephemeral volume component and let me know what you think, thanks!

Copy link

@abrennan89 abrennan89 left a comment

Choose a reason for hiding this comment

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

Added some comments

Copy link
Contributor

@kowen-rh kowen-rh left a comment

Choose a reason for hiding this comment

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

Just a couple of minor nitpicks that I happened to notice. Once these are addressed, this PR looks good to me!

@openshift-ci
Copy link

openshift-ci bot commented Feb 11, 2022

@robin-owen: changing LGTM is restricted to collaborators

In response to this:

Just a couple of minor nitpicks that I happened to notice. Once these are addressed, this PR looks good to me!

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@openshift-ci openshift-ci bot removed the lgtm label Feb 13, 2022
jc-berger and others added 9 commits February 14, 2022 14:09
…-component-to-a-devfile.adoc

Co-authored-by: Robert Krátký <rkratky@redhat.com>
…-component-to-a-devfile.adoc

Co-authored-by: Robert Krátký <rkratky@redhat.com>
…penshift-component-to-a-devfile.adoc

Co-authored-by: Robert Krátký <rkratky@redhat.com>
…penshift-component-to-a-devfile.adoc

Co-authored-by: Robert Krátký <rkratky@redhat.com>
…evfile.adoc

Co-authored-by: Robert Krátký <rkratky@redhat.com>
…evfile.adoc

Co-authored-by: Robert Krátký <rkratky@redhat.com>
@openshift-ci openshift-ci bot added the lgtm label Feb 21, 2022
@openshift-ci
Copy link

openshift-ci bot commented Feb 21, 2022

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: jc-berger, kim-tsao, rkratky

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@rkratky rkratky merged commit edac749 into devfile:master Feb 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation lgtm

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Complete list of components

5 participants