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

ci: docker release login with dianwen #47

Merged
merged 1 commit into from
Aug 26, 2022
Merged

Conversation

dianwen
Copy link
Member

@dianwen dianwen commented Aug 26, 2022

Description

There is a permissions issue pushing images using the production account, but the dianwen account has been added as a collaborator, so just authenticate with that account.

Type of change

  • 🐛 Bug fix (non-breaking change which fixes an issue)
  • 😎 New feature (non-breaking change which adds functionality)
  • ⁉️ Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • ⚒️ Refactor (no functional changes)
  • 📖 Documentation (updating or adding docs)

How Has This Been Tested?

Please describe the tests that you ran to verify your changes.

@dianwen dianwen requested review from jonathankau and brianluong and removed request for jonathankau August 26, 2022 22:38
@dianwen dianwen merged commit 15fb125 into main Aug 26, 2022
@dianwen dianwen deleted the release-login-with-dianwen branch August 26, 2022 22:46
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