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

feat: sign with keyless approach #76

Merged
merged 1 commit into from
Jan 18, 2022
Merged

Conversation

developer-guy
Copy link
Collaborator

Signed-off-by: Batuhan Apaydın batuhan.apaydin@trendyol.com

Fixes #71

Copy link
Collaborator

@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.

looks cool
can you change here as well https://github.com/gythialy/golang-cross/blob/main/.github/workflows/builder.yml#L41 we sign the builder image too :)

@developer-guy
Copy link
Collaborator Author

looks cool can you change here as well main/.github/workflows/builder.yml#L41 we sign the builder image too :)

Thanks, @cpanato, you always have my back. 🤝

Copy link
Collaborator

@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.

thanks for working on this
just a small note

.github/workflows/builder.yml Show resolved Hide resolved
Signed-off-by: Batuhan Apaydın <batuhan.apaydin@trendyol.com>
Copy link
Collaborator

@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.

i think we are good
thanks for working on this!

@cpanato cpanato merged commit 0dc6721 into gythialy:main Jan 18, 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.

sign with keyless approach
2 participants