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

Add support for other registries and bump dependencies #9

Merged
merged 2 commits into from Sep 16, 2020

Conversation

TrueBrain
Copy link
Member

@TrueBrain TrueBrain commented Sep 16, 2020

This will be v2, as it breaks compatibility with v1.

Docker Hub

See the next two URLs for proof it still works (this was a run with unmodified GitHub workflows):
https://github.com/TrueBrain/test-actions/actions/runs/257943006
https://github.com/TrueBrain/test-actions/actions/runs/257950716

With the resulting image here:
docker pull truebrain/test-actions:development
docker pull docker.io/truebrain/test-actions:development

(yes, the last one (deliberately) fails, and by then all the actions are triggered at least once)

GitHub Container Registry (GHCR)

And to proof a different registry works (this was a run with modified GitHub workflows):
https://github.com/TrueBrain/test-actions/actions/runs/257965556

With the resulting image here:
docker pull ghcr.io/truebrain/test-actions:development

This mainly resolves a CVE warning in node-fetch.
@TrueBrain TrueBrain force-pushed the more_registries branch 5 times, most recently from debb053 to f85084d Compare September 16, 2020 18:35
This allows migration to the GitHub Container Registry (ghcr.io).
@TrueBrain TrueBrain merged commit f005561 into OpenTTD:master Sep 16, 2020
@TrueBrain TrueBrain deleted the more_registries branch September 16, 2020 18:43
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.

None yet

2 participants