-
Notifications
You must be signed in to change notification settings - Fork 505
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: enhance Docker publish workflow with additional tagging options #160
fix: enhance Docker publish workflow with additional tagging options #160
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.
Comments suppressed due to low confidence (1)
.github/workflows/docker-publish.yml:79
- The 'type=match' rule does not include a pattern or additional configuration, which may lead to unintended matching behavior. Please review and add the necessary parameters if a specific pattern is intended.
type=match
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.
Thanks for this, I removed the match because it requires a pattern to be provided, but this is a big improvement on the existing tags. |
This pull request includes a significant update to the
docker-publish.yml
workflow file. The primary change is the addition of multiple tagging rules for Docker images, which enhances the flexibility and control over how images are tagged during the publishing process.Enhancements to Docker image tagging:
.github/workflows/docker-publish.yml
: Added various tagging rules including schedule-based, branch-based, tag-based, pull request-based, semantic versioning patterns, SHA, edge, match, and PEP 440. Additionally, a custom rule was added to prevent pre-releases from getting the "latest" tag.Closes #156`