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

fix: update travis CI Docker image repository #930

Merged
merged 1 commit into from
May 14, 2022

Conversation

cokia
Copy link
Contributor

@cokia cokia commented May 13, 2022

based on issue #929

Hello.

When I checked the Travis CI now, I checked the case where it was failed due to DockerHub's Image Pull Rate Limit.

I suggest using awsecr public gallery where official nodejs image is provided.

If this is used, it is not restricted by Rate Limit, and the test can be carried out using the same image as the official Docker image.

@recrsn recrsn merged commit 5ce7348 into kelektiv:master May 14, 2022
@cokia cokia deleted the docker-repository-change branch May 14, 2022 09:15
fast-facts pushed a commit to fast-facts/node.bcrypt.js that referenced this pull request Jun 17, 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.

2 participants