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

Sign releases, dev builds, and nuget package #4189

Merged
merged 4 commits into from
Sep 22, 2024

Conversation

HebaruSan
Copy link
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
Copy link
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
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
3 checks passed
@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