Skip to content

Conversation

@rquitales
Copy link
Contributor

Proposed changes

This PR adds a new Makefile target make sign-goreleaser-exe target to sign all built GoReleaser windows binaries. This PR contains 2 changes:

Please see the linked ci-mgmt issue for status of GitHub actions workflows to validate that the binaries are signed.

@rquitales rquitales requested review from a team and EronWright January 27, 2025 19:03
@rquitales rquitales self-assigned this Jan 27, 2025
@codecov
Copy link

codecov bot commented Jan 27, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 41.15%. Comparing base (d05645f) to head (671ba17).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3455      +/-   ##
==========================================
+ Coverage   41.13%   41.15%   +0.02%     
==========================================
  Files          87       87              
  Lines       12910    12910              
==========================================
+ Hits         5310     5313       +3     
+ Misses       7205     7203       -2     
+ Partials      395      394       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@github-actions
Copy link

Does the PR have any schema changes?

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

@rquitales rquitales added the impact/no-changelog-required This issue doesn't require a CHANGELOG update label Jan 27, 2025
@rquitales
Copy link
Contributor Author

Note: our prerelease workflow is failing because the current major version is v4, but the pre-release tag used is v0.0.1-alpha.101. This causes issues with pulling the right versions with pulumi/publish-go-sdk-action.

@rquitales rquitales merged commit b776b68 into master Jan 27, 2025
35 of 36 checks passed
@rquitales rquitales deleted the rquitales/add-windows-signing-make-target branch January 27, 2025 20:53
rquitales added a commit that referenced this pull request Jan 28, 2025
### Added

- Windows executable releases are now signed.
(<#3455>)
@pulumi-bot
Copy link
Contributor

This PR has been shipped in release v4.21.1.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

impact/no-changelog-required This issue doesn't require a CHANGELOG update

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants