-
Notifications
You must be signed in to change notification settings - Fork 1.4k
Added support for validating OCI tags #1372
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
base: master
Are you sure you want to change the base?
Conversation
Thank you for your contribution! |
I've updated my branch with latest changes from upstream and added to the README.md and doc.go file. |
@@ -211,6 +215,7 @@ validate := validator.New(validator.WithRequiredStructEnabled()) | |||
| uuid_rfc4122 | Universally Unique Identifier UUID RFC4122 | | |||
| md4 | MD4 hash | | |||
| md5 | MD5 hash | | |||
| oci_tag | Open Container Initiative (OCI) Tag | |
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.
Can we maybe leave this and remove all the other formatting changes?
Otherwise, it doesn't feel clear and should probably be cleaned up in a different PR. There are small things like some lost/changed formatting here and there that I would rather not make part of this PR.
Fixes Or Enhances
Added a new
oci_tag
to validate an OCI manifest tag as described in the OCI Distribution Spec: https://github.com/opencontainers/distribution-spec/blob/main/spec.md?plain=1#L164-L166This is also more commonly referred to as a container tag or a docker tag.
Make sure that you've checked the boxes below before you submit PR:
@go-playground/validator-maintainers