Skip to content

Conversation

@yangcao77
Copy link
Contributor

Signed-off-by: Stephanie [email protected]

Please specify the area for this PR

What does does this PR do / why we need it:
add docs check in PR acceptance criteria

Which issue(s) this PR fixes:

Fixes devfile/api#839

PR acceptance criteria:

  • Test (WIP)

  • Documentation (WIP)

How to test changes / Special notes to the reviewer:

@yangcao77 yangcao77 requested review from johnmcollier and kim-tsao May 6, 2022 15:00
@openshift-ci openshift-ci bot requested review from elsony and maysunfaisal May 6, 2022 15:00
@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label May 10, 2022
@openshift-ci
Copy link

openshift-ci bot commented May 10, 2022

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: kim-tsao, yangcao77
To complete the pull request process, please assign maysunfaisal after the PR has been reviewed.
You can assign the PR to them by writing /assign @maysunfaisal in a comment when ready.

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

@yangcao77 yangcao77 merged commit 192202e into devfile:main May 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update PR template in registry-support to add check to document update

2 participants