Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat: Add env variable to facilitate CI tests of Action installer #393

Merged
merged 3 commits into from
Dec 6, 2022

Conversation

laurentsimon
Copy link
Contributor

Signed-off-by: laurentsimon laurentsimon@google.com

@laurentsimon laurentsimon requested review from kpk47 and asraa December 6, 2022 20:13
Signed-off-by: laurentsimon <laurentsimon@google.com>
Signed-off-by: laurentsimon <laurentsimon@google.com>
Signed-off-by: laurentsimon <laurentsimon@google.com>
@laurentsimon laurentsimon enabled auto-merge (squash) December 6, 2022 20:17
@laurentsimon
Copy link
Contributor Author

@asraa let's also include this PR in the next release. This will allow me to tests the Action in a loop for all releases.

Copy link
Contributor

@asraa asraa left a comment

Choose a reason for hiding this comment

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

sg -- i'll wait on the merge and kick off a RC

@laurentsimon laurentsimon merged commit 4cba39a into slsa-framework:main Dec 6, 2022
ramonpetgrave64 pushed a commit to ramonpetgrave64/slsa-verifier that referenced this pull request Apr 18, 2024
* golanglintci settings

These settings are based on https://raw.githubusercontent.com/ossf/scorecard/main/.golangci.yml

Signed-off-by: naveensrinivasan <172697+naveensrinivasan@users.noreply.github.com>

* More tweaks to the linter

- Removed deprecated linters for 1.18
- Fixed few linter issues using the `fix`

Signed-off-by: naveensrinivasan <172697+naveensrinivasan@users.noreply.github.com>

Co-authored-by: Ian Lewis <ianlewis@google.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants