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.

Sign up for GitHub

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

Jump to bottom

[ci] Make publish docker action more secure #1525

Merged

alvicsam merged 1 commit into master from as-fix-ci

Mar 7, 2024

Contributor

alvicsam commented Mar 7, 2024

Summary

Closes #_

n | Does it introduce breaking changes?
n | Is it dependent on the specific version of ink or pallet-contracts?

More info can be found in this issue: https://github.com/paritytech/ci_cd/issues/957

Description

Checklist before requesting a review

My code follows the style guidelines of this project
I have added an entry to CHANGELOG.md
I have commented my code, particularly in hard-to-understand areas
I have added tests that prove my fix is effective or that my feature works
Any dependent changes have been merged and published in downstream modules


          [ci] Make publish docker action more secure

66a3a47

alvicsam requested review from cmichi, ascjones, SkymanOne and a team as code owners

March 7, 2024 09:47

rcny approved these changes

View reviewed changes

lovelaced approved these changes

View reviewed changes

alvicsam merged commit 077b36e into master

10 checks passed

alvicsam deleted the as-fix-ci branch

March 7, 2024 09:49

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

rcny rcny approved these changes

lovelaced lovelaced approved these changes

cmichi Awaiting requested review from cmichi cmichi is a code owner

ascjones Awaiting requested review from ascjones ascjones is a code owner

SkymanOne Awaiting requested review from SkymanOne

Labels

None yet