Skip to content

Sign releases, dev builds, and nuget package#4189

Merged
HebaruSan merged 4 commits into
KSP-CKAN:masterfrom
HebaruSan:feature/signing
Sep 22, 2024
Merged

Sign releases, dev builds, and nuget package#4189
HebaruSan merged 4 commits into
KSP-CKAN:masterfrom
HebaruSan:feature/signing

Conversation

@HebaruSan
Copy link
Copy Markdown
Member

Motivation

SignPath's action is ready, we have a certificate, and #4188 created a helper build target to collect assets for signing. The next step toward signing releases is updating our workflows.

Changes

  • Now build.yml uses the build target from Add Prepare-SignPath target to build script #4188 to upload an artifact containing the assets we need to sign for the Release configuration
  • Now sign.yml is created to retrieve that artifact, get it signed, and upload a new signed artifact with the results
  • Now release.yml calls sign.yml to generate signed assets for releases
  • Now deploy.yml calls sign.yml to generate signed assets for uploading dev builds

@HebaruSan HebaruSan added Build Issues affecting the build system Infrastructure Issues affecting everything around CKAN (the GitHub repos, build process, CI, ...) labels Sep 21, 2024
@HebaruSan HebaruSan requested a review from techman83 September 21, 2024 21:43
Copy link
Copy Markdown
Member

@techman83 techman83 left a comment

Choose a reason for hiding this comment

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

Only change will be, can we update the Readme. Otherwise all looks good to me!

Copy link
Copy Markdown
Member

@techman83 techman83 left a comment

Choose a reason for hiding this comment

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

Love it! Thanks @HebaruSan

@HebaruSan HebaruSan merged commit b783be9 into KSP-CKAN:master Sep 22, 2024
@HebaruSan HebaruSan deleted the feature/signing branch September 22, 2024 03:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Build Issues affecting the build system Infrastructure Issues affecting everything around CKAN (the GitHub repos, build process, CI, ...)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants