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

Use docker/login-action rather than aws-actions/amazon-ecr-login #28

Merged
merged 1 commit into from
Apr 19, 2022

Conversation

Yoshiki-Iwasa
Copy link
Contributor

@Yoshiki-Iwasa Yoshiki-Iwasa commented Apr 19, 2022

use docker/login-action because aws-actions/amazon-ecr-login seems not to be updated.

@@ -43,7 +43,9 @@ jobs:
role-to-assume: ${{ secrets.IAM_ROLE }}
aws-region: ap-northeast-1
- name: Login to Amazon ECR
uses: aws-actions/amazon-ecr-login@v1
uses: docker/login-action@v1
with:

Choose a reason for hiding this comment

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

[actionlint] reported by reviewdog 🐶
could not parse as YAML: yaml: line 47: mapping values are not allowed in this context [yaml-syntax]

@Yoshiki-Iwasa Yoshiki-Iwasa changed the title docker-login を使用 use docker-login Apr 19, 2022
@KOBA789 KOBA789 changed the title use docker-login Use docker/login-action rather than aws-actions/amazon-ecr-login Apr 19, 2022
@Yoshiki-Iwasa Yoshiki-Iwasa force-pushed the fix/replace-aws-actions/amazon-ecr-login branch from 5965ddd to 3ff1184 Compare April 19, 2022 06:51
@Yoshiki-Iwasa Yoshiki-Iwasa requested a review from sksat April 19, 2022 07:00
@Yoshiki-Iwasa Yoshiki-Iwasa force-pushed the fix/replace-aws-actions/amazon-ecr-login branch from 3ff1184 to 23766b5 Compare April 19, 2022 07:07
@sksat sksat merged commit 9b2f937 into main Apr 19, 2022
@sksat sksat deleted the fix/replace-aws-actions/amazon-ecr-login branch April 19, 2022 15:12
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