Skip to content

feat: Action to compute a sha256 of a file#436

Merged
laurentsimon merged 5 commits intoslsa-framework:mainfrom
laurentsimon:feat/compute-sha-action
Jun 30, 2022
Merged

feat: Action to compute a sha256 of a file#436
laurentsimon merged 5 commits intoslsa-framework:mainfrom
laurentsimon:feat/compute-sha-action

Conversation

@laurentsimon
Copy link
Collaborator

@laurentsimon laurentsimon commented Jun 29, 2022

This is part of #26

This PR defines an action to compute the sha256 of a file.

I will send follow-up PRs to use these it as part of artifact verification and for other uses in the re-usable workflows. We may later convert this code to TS or another programming language.

@ianlewis
Copy link
Member

How much value do we get out of just computing the sha256? I thought maybe our action would combine the generate sha and upload steps. And another action could combine the download and verify steps. I'm curious what you had in mind?

@laurentsimon laurentsimon force-pushed the feat/compute-sha-action branch from 395dc13 to 18af68d Compare June 29, 2022 22:41
@laurentsimon laurentsimon enabled auto-merge (squash) June 29, 2022 22:41
@laurentsimon laurentsimon disabled auto-merge June 30, 2022 00:43
@laurentsimon laurentsimon enabled auto-merge (squash) June 30, 2022 00:47
@laurentsimon laurentsimon force-pushed the feat/compute-sha-action branch from a3a42c2 to 18af68d Compare June 30, 2022 00:51
@laurentsimon laurentsimon force-pushed the feat/compute-sha-action branch from 18af68d to fa389af Compare June 30, 2022 00:52
@laurentsimon laurentsimon disabled auto-merge June 30, 2022 00:54
@laurentsimon laurentsimon enabled auto-merge (squash) June 30, 2022 00:55
@laurentsimon laurentsimon disabled auto-merge June 30, 2022 01:00
@laurentsimon laurentsimon merged commit aaaf11b into slsa-framework:main Jun 30, 2022
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.

2 participants