Skip to content

feat: push to ghcr#268

Merged
FedericoPonzi merged 3 commits intoFedericoPonzi:masterfrom
kemingy:ghcr
Feb 27, 2025
Merged

feat: push to ghcr#268
FedericoPonzi merged 3 commits intoFedericoPonzi:masterfrom
kemingy:ghcr

Conversation

@kemingy
Copy link
Copy Markdown
Collaborator

@kemingy kemingy commented Feb 27, 2025

  • push to GitHub package due to the upcoming Docker Hub pull limit
  • remove unused dependencies
  • add platform linux/arm64

Motivation and Context

Description

How Has This Been Tested?

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Performance enhancement (non-breaking change which improves efficiency)
  • Code cleanup (non-breaking change which makes code smaller or more readable)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation (a change to man pages or other documentation)

Checklist:

  • I have updated the documentation accordingly.
  • I have added tests to cover my changes.

Signed-off-by: Keming <kemingy94@gmail.com>
Signed-off-by: Keming <kemingy94@gmail.com>
Signed-off-by: Keming <kemingy94@gmail.com>
@FedericoPonzi
Copy link
Copy Markdown
Owner

Thanks for the pr(s)! I wasn't aware of this upcoming change but reading on https://www.theregister.com/2025/02/22/docker_hub_pull_limits/ it looks like it should not affect the push of containers, right? I'm thinking that in order to keep backward compatibility, we could keep pushing to docker registry and additionally push to ghcr in case someone would prefer it. What do you think?

@kemingy
Copy link
Copy Markdown
Collaborator Author

kemingy commented Feb 27, 2025

Yes, this will push to both dockerhub and ghcr.

So the generated tag will contain docker.io (omit) & ghcr.io.

@FedericoPonzi FedericoPonzi merged commit ea16bc6 into FedericoPonzi:master Feb 27, 2025
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