Skip to content

add CI test that exercises auth#38

Merged
imjasonh merged 4 commits intochainguard-dev:mainfrom
imjasonh:ci-auth
Dec 7, 2025
Merged

add CI test that exercises auth#38
imjasonh merged 4 commits intochainguard-dev:mainfrom
imjasonh:ci-auth

Conversation

@imjasonh
Copy link
Contributor

@imjasonh imjasonh commented Dec 5, 2025

blocked on setting up the identity

Signed-off-by: Jason Hall <[email protected]>
@imjasonh imjasonh marked this pull request as ready for review December 7, 2025 02:45
@imjasonh imjasonh requested review from cpanato and mattmoor December 7, 2025 02:45
Copy link
Member

@cpanato cpanato left a comment

Choose a reason for hiding this comment

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

non blocking question

push:
branches:
- 'main'
pull_request_target:
Copy link
Member

Choose a reason for hiding this comment

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

as this repo is public, are we ok having this?

or run just on a post submit?

Copy link
Member

Choose a reason for hiding this comment

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

I think it's fine. The identity exists, but doesn't have any access grants, so it is pretty low risk.

cc @eslerm FYI

@imjasonh imjasonh merged commit a93b3f9 into chainguard-dev:main Dec 7, 2025
6 checks passed
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.

3 participants