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

Regenerate Github Actions workflows for pulumi-acme #95

Merged
merged 2 commits into from
Dec 10, 2024

Conversation

ringods
Copy link
Member

@ringods ringods commented Dec 10, 2024

This pull request was generated automatically by the resync-build workflow in this repository.

Copy link
Contributor

Does the PR have any schema changes?

Looking good! No breaking changes found.
No new resources/functions.

Maintainer note: consult the runbook for dealing with any breaking changes.

simenandre
simenandre previously approved these changes Dec 10, 2024
@ringods
Copy link
Member Author

ringods commented Dec 10, 2024

@simenandre why approve if the linked run fails? The Pulumi team integrated signing their Windows binary, but we don't have that configured, so it fails for Pulumiverse builds.

@simenandre
Copy link
Member

@simenandre why approve if the linked run fails? The Pulumi team integrated signing their Windows binary, but we don't have that configured, so it fails for Pulumiverse builds.

Why should I review if a run fails?

@ringods ringods force-pushed the chore/resync-12248868650 branch from e86a916 to 16ad22e Compare December 10, 2024 08:09
@ringods ringods requested a review from simenandre December 10, 2024 08:23
@ringods
Copy link
Member Author

ringods commented Dec 10, 2024

@simenandre as far as I know, I did not request for a review.

My last commit fixes the workflows again and I submitted a fix upstream. So this PR is good for review (and approval 😄 ).

@ringods ringods merged commit 9cec5ff into main Dec 10, 2024
20 checks passed
@ringods ringods deleted the chore/resync-12248868650 branch December 10, 2024 10:25
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.

3 participants