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

schema: add ibmcloud to schema #1740

Merged
merged 2 commits into from
Sep 24, 2020

Conversation

dustymabe
Copy link
Member

No description provided.

@dustymabe
Copy link
Member Author

cc @darkmuggle for a sanity check

@@ -183,6 +183,7 @@
"pkgdiff",
"parent-pkgdiff",
"release-payload",
"ibmcloud",
Copy link
Member

Choose a reason for hiding this comment

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

nit: alphabetize the list?

Copy link
Member Author

Choose a reason for hiding this comment

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

added a second commit to make it alphabetical

@miabbott
Copy link
Member

Minor nit, but otherwise LGTM

/approve

src/schema/v1.json Outdated Show resolved Hide resolved
Copy link
Contributor

@darkmuggle darkmuggle left a comment

Choose a reason for hiding this comment

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

/lgtm

Excellent work.

@openshift-ci-robot
Copy link

New changes are detected. LGTM label has been removed.

@dustymabe
Copy link
Member Author

I did a force push to update the CI job for this to also build the IBM Cloud image.

Copy link
Contributor

@darkmuggle darkmuggle left a comment

Choose a reason for hiding this comment

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

/lgtm still

@openshift-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: darkmuggle, dustymabe, miabbott

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:
  • OWNERS [darkmuggle,dustymabe,miabbott]

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

@dustymabe dustymabe added the lgtm label Sep 24, 2020
@openshift-merge-robot openshift-merge-robot merged commit 2ed9c96 into coreos:master Sep 24, 2020
jlebon added a commit to jlebon/coreos-assembler that referenced this pull request Sep 25, 2020
Otherwise it's not in the golang generated code.

Follow-up to coreos#1740.
openshift-merge-robot pushed a commit that referenced this pull request Sep 25, 2020
Otherwise it's not in the golang generated code.

Follow-up to #1740.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants