Skip to content

Conversation

@sampaiodiego
Copy link
Member

@sampaiodiego sampaiodiego commented Oct 19, 2020

https://docs.github.com/en/free-pro-team@latest/packages/getting-started-with-github-container-registry/migrating-to-github-container-registry-for-docker-images

Proposed changes

  • Uses docker/login-action@v1 instead of docker login
  • Removed unneeded export keywords to make file look cleaner

Issue(s)

How to test or reproduce

Screenshots

Types of changes

Checklist

Changelog

Further comments

@sampaiodiego sampaiodiego marked this pull request as ready for review October 21, 2020 06:02
@sampaiodiego
Copy link
Member Author

Yay! 🎉 I finally could get the right permissions set up.

you guys can review this now. the new image has now public access and is published at https://github.com/orgs/RocketChat/packages/container/package/rocket.chat

@rodrigok
Copy link
Member

@sampaiodiego very nice improvement :)

@sampaiodiego sampaiodiego merged commit 3b06de0 into develop Oct 23, 2020
@sampaiodiego sampaiodiego deleted the use-github-container-registry branch October 23, 2020 13:59
@sampaiodiego sampaiodiego mentioned this pull request Nov 14, 2020
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