Skip to content

Conversation

@rcgoodfellow
Copy link
Collaborator

Make it so that CI is run for pull requests. This is helpful when reviewing request and for PR submitters to see the CI rendering of spec changes.

@rcgoodfellow rcgoodfellow force-pushed the build-spec-on-pr branch 2 times, most recently from 051da02 to 3836e31 Compare March 31, 2025 17:51
Signed-off-by: Ryan Goodfellow <ryan.goodfellow@oxide.computer>
Copy link
Collaborator

@jonathan-dilorenzo jonathan-dilorenzo left a comment

Choose a reason for hiding this comment

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

This is great! Is there somewhere I can see the spec associated e.g. with this PR (even though it would be unchanged) just to see an example of this in action?

@rcgoodfellow
Copy link
Collaborator Author

Is there somewhere I can see the spec associated e.g. with this PR

yeah, if you click on one of the successful jobs with the green checkmark next to it, and then click on "summary" in the upper left hand part of the web page, you should be able to download the CI produced artifacts.

@jonathan-dilorenzo
Copy link
Collaborator

Super cool. Merging!

@jonathan-dilorenzo jonathan-dilorenzo merged commit 7f4dbd4 into main Mar 31, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants