Skip to content

DevTools Week 09/22: Docs Update - Devfile Registry #36

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 8 commits into from
Mar 31, 2023

Conversation

michael-valdron
Copy link
Member

@michael-valdron michael-valdron commented Sep 15, 2022

What does this PR do / why we need it:

Revisions done for the Devfile Registry section of the doc pages.

Changes

  • building-a-custom-devfile-registry.md
    • golang version updated
    • Dockerfile instruction updated
    • air gap environment manual added as additional resource
  • creating-a-devfile-stack.md
    • Procedure steps 2 expanded
  • deploying-a-devfile-registry.md
    • Deploying the devfile registry on Kubernetes cluster formatting fixed
  • installation-of-in-cluster-offline-devfile-registry.md
    • Modify devfile verification step heading fixed
  • adding-a-stack-yaml-file.md
  • adding-a-registry-schema.md
    • example headers fixed

Which issue(s) this PR fixes:

Fixes #?

fixes devfile/api#1090

PR acceptance criteria:

  • Unit Tests
  • E2E Tests
  • Documentation

How to test changes / Special notes to the reviewer:

@nx-cloud
Copy link

nx-cloud bot commented Sep 15, 2022

☁️ Nx Cloud Report

CI is running/has finished running commands for commit 133d0bd. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this branch


✅ Successfully ran 5 targets

Sent with 💌 from NxCloud.

@michael-valdron michael-valdron changed the title DevTools Week Docs Update: Devfile Registry DevTools Week 09/22: Docs Update - Devfile Registry Mar 31, 2023
@openshift-ci openshift-ci bot added the lgtm label Mar 31, 2023
@openshift-ci
Copy link

openshift-ci bot commented Mar 31, 2023

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: michael-valdron, mike-hoang

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

The pull request process is described 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

- golang version updated
- air gap environment manual added as additional resource

Signed-off-by: Michael Valdron <mvaldron@redhat.com>
Signed-off-by: Michael Valdron <mvaldron@redhat.com>
…egistry on Kubernetes cluster formatting fixed.

Signed-off-by: Michael Valdron <mvaldron@redhat.com>
- Modify devfile verification step heading fixed
- Colon at the end of modify devfile overview paragraph replace with period

Signed-off-by: Michael Valdron <mvaldron@redhat.com>
Signed-off-by: Michael Valdron <mvaldron@redhat.com>
Signed-off-by: Michael Valdron <mvaldron@redhat.com>
Signed-off-by: Michael Valdron <mvaldron@redhat.com>
Signed-off-by: Michael Valdron <mvaldron@redhat.com>
@openshift-ci
Copy link

openshift-ci bot commented Mar 31, 2023

New changes are detected. LGTM label has been removed.

@michael-valdron michael-valdron merged commit caac377 into devfile:main Mar 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Revisions to the devfile registry doc pages
4 participants