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

chore: Add GH action and release please support #4

Merged
merged 1 commit into from
Apr 29, 2024

Conversation

keelerm84
Copy link
Member

No description provided.

@keelerm84 keelerm84 requested a review from a team April 26, 2024 14:11
.github/actions/build/action.yml Outdated Show resolved Hide resolved
contents: write
uses: slsa-framework/slsa-github-generator/.github/workflows/generator_generic_slsa3.yml@v1.10.0
with:
base64-subjects: "${{ needs.build-publish.outputs.package-hashes }}"

Choose a reason for hiding this comment

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

Curious why this is quoted? is it an array or something interesting?

Copy link
Member Author

Choose a reason for hiding this comment

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

I'm not sure. Rouel did the original provenance work. I've just been applying his templates to my other repos. My guess is just protecting from some weird characters or just being overly cautious.

upload-tag-name: ${{ steps.release.outputs.tag_name }}
package-hashes: ${{ steps.build.outputs.package-hashes}}
steps:
- uses: google-github-actions/release-please-action@v3

Choose a reason for hiding this comment

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

Would using @v4 work?

Copy link
Member Author

Choose a reason for hiding this comment

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

Maybe. Ryan had a similar question on the ruby otel branch. I'm just keeping my repos all using the same version of release please for now, and then I'll go through and update them in mass. This way it's a one and done thing. It also let's this be a copy-paste job, so I'm confident the first release will work.

@keelerm84 keelerm84 force-pushed the mk/sc-241885/support-documents branch from 571f137 to 537819e Compare April 26, 2024 19:28
Base automatically changed from mk/sc-241885/support-documents to main April 26, 2024 19:31
@keelerm84 keelerm84 force-pushed the mk/sc-241885/gh-actions branch 2 times, most recently from 08cea4b to 269715f Compare April 26, 2024 19:35
@keelerm84 keelerm84 merged commit 5584b32 into main Apr 29, 2024
10 checks passed
@keelerm84 keelerm84 deleted the mk/sc-241885/gh-actions branch April 29, 2024 18:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants