Skip to content

added doc for how to develop with devfiles #119

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

Merged
merged 20 commits into from
Jan 21, 2022
Merged

added doc for how to develop with devfiles #119

merged 20 commits into from
Jan 21, 2022

Conversation

jc-berger
Copy link
Contributor

Fixes devfile/api#380

Fixes https://issues.redhat.com/browse/RHDEVDOCS-2869

Looking at #380, this PR does not address all 3 points, but it does provide "how to" instructions for how to develop a simple application by using devfiles.

Let me know if you think this is a good start for beginning "how to" documentation for devfiles.

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

github-actions bot commented Jan 5, 2022

⚡️ Deploying PR Preview...

Copy link
Contributor

@yangcao77 yangcao77 left a comment

Choose a reason for hiding this comment

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

The change looks good to me. I'm fine with having two separate push cmds if want to split the initial build and the application update build for URL creation

@missmesss
Copy link
Contributor

@jc-berger The procedure looks amazing, thank you. I'm only adding a couple of suggestions.

@openshift-ci openshift-ci bot removed the lgtm label Jan 12, 2022
@jc-berger
Copy link
Contributor Author

@rkratky please take a look and merge if you see fit, thanks!

@openshift-ci
Copy link

openshift-ci bot commented Jan 12, 2022

@missmesss: changing LGTM is restricted to collaborators

In response to this:

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.

Copy link

@rkratky rkratky left a comment

Choose a reason for hiding this comment

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

@jc-berger, comments added.

@jc-berger
Copy link
Contributor Author

@rkratky thanks! Ready for your second look!

@openshift-ci openshift-ci bot added the lgtm label Jan 21, 2022
@openshift-ci
Copy link

openshift-ci bot commented Jan 21, 2022

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: jc-berger, missmesss, rkratky, yangcao77

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 1776809 into devfile:master Jan 21, 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.

How-tos document to help user to understand how to use devfile in different scenarios
5 participants