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

ci(pr-build): integrate check runs to report build status #162

Merged
merged 8 commits into from
Jul 14, 2023
Merged

Conversation

piyoki
Copy link
Contributor

@piyoki piyoki commented Jul 14, 2023

Background

Integrate check runs to report build status.

Relevant workflows:

References:

Checklist

Full changelogs

  • ci(pr-build): integrate check runs to report build status
  • ci(pr-build): integrate individual check run per stage
  • chore: add title for check-run step
  • ci(pr-build): add instantiate-check-runs stage

Issue reference

Closes #161

Test Result

https://github.com/daeuniverse/daed/actions/runs/5549917316

image image CleanShot 2023-07-14 at 11 12 15@2x CleanShot 2023-07-14 at 11 21 34@2x

@piyoki
Copy link
Contributor Author

piyoki commented Jul 14, 2023

The proposed solution has already been tested. Ready for review.

@piyoki piyoki added the tested label Jul 14, 2023
dae-prow[bot]
dae-prow bot previously approved these changes Jul 14, 2023
Copy link
Contributor

@dae-prow dae-prow bot left a comment

Choose a reason for hiding this comment

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

🧪 Since the PR has been fully tested, please consider merging it.

kunish
kunish previously approved these changes Jul 14, 2023
Copy link
Member

@kunish kunish left a comment

Choose a reason for hiding this comment

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

Great, thank you for your efforts

@piyoki piyoki dismissed stale reviews from kunish and dae-prow via dbff16d July 14, 2023 03:00
@piyoki
Copy link
Contributor Author

piyoki commented Jul 14, 2023

Working like a charm 🤟

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.

[CI] Add auto-merge support for sync-upstream workflow
2 participants