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

📦 Update workflow with configure_trusted_publisher #280

Merged
merged 1 commit into from
May 18, 2024

Conversation

nevans
Copy link
Collaborator

@nevans nevans commented May 18, 2024

I ran configure_trusted_publisher to regenerate and configure trusted publishing, and copied over most of what it created. The only significant difference is that it adds step-security/harden-runner. The other changes are minor: renaming the workflow file and the GitHub environment, using SHA1 checksums for the actions rather than tags, etc.

configure_trusted_publisher originally configured the trusted publisher on rubygems.org without any reference to the GitHub environment. But then I made the following PR: rubygems/configure_trusted_publisher#9, and used that to generate a new RubyGems Trusted Publisher config.

I ran `configure_trusted_publisher` to regenerate and configure trusted
publishing, and copied over most of what it created.  The only
significant difference is that it adds `step-security/harden-runner`.
The other changes are minor: renaming the workflow file and the GitHub
environment, using SHA1 checksums for the actions rather than tags, etc.

`configure_trusted_publisher` originally configured the trusted
publisher on rubygems.org _without_ any reference to the GitHub
environment.  But then I made the following PR:
rubygems/configure_trusted_publisher#9, and used that to generate a new
RubyGems Trusted Publisher config.
@nevans nevans force-pushed the configure_trusted_publisher branch from 2946a66 to 80c8964 Compare May 18, 2024 18:54
@nevans nevans merged commit 80c8964 into master May 18, 2024
26 checks passed
@nevans nevans deleted the configure_trusted_publisher branch May 18, 2024 18:59
@nevans
Copy link
Collaborator Author

nevans commented Jun 1, 2024

@shugo I think this failed for v0.4.12 because I accidentally configured the environment to require a branch matching v*. I've updated it to work with any tags matching v*. So hopefully we'll be more lucky with v0.4.13 and v0.5.0. 🙂

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

Successfully merging this pull request may close these issues.

1 participant