Skip to content

Conversation

@mattyclarkson
Copy link
Contributor

umoci modifies Open Container images.

umoci (pronounced /uːmoˈʨi/ or approximately "oo-mo-tchee") is a reference implementation of the OCI image specification and provides users with the ability to create, manipulate, and otherwise interact with container images. It is designed to be as small and unopinonated as possible, so as to act as a foundation for larger systems to be built on top of.

@bazel-io
Copy link
Member

Hello @bazelbuild/bcr-maintainers, modules without existing maintainers (umoci) have been updated in this PR.
Please review the changes. You can view a diff against the previous version in the "Generate module diff" check.

Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request introduces the umoci module at version 0.6.0. The submission is well-structured and adheres to the Bazel Central Registry (BCR) style guide. It includes all necessary files such as MODULE.bazel, metadata.json, and presubmit.yml. The use of a BUILD file overlay for this Go project is properly documented in the included README.md, and the presubmit configuration correctly defines tasks to verify the build targets. The overall quality of this contribution is high.

@meteorcloudy
Copy link
Member

See also #6230 (comment)

Not a big fan of checking so many BUILD files in BCR, if you want to make those go binaries bazel module, maybe it's better to have a specific bazel Go registry or something. /cc @fmeum

@mattyclarkson
Copy link
Contributor Author

See also #6230 (comment)

Not a big fan of checking so many BUILD files in BCR, if you want to make those go binaries bazel module, maybe it's better to have a specific bazel Go registry or something. /cc @fmeum

Good with me. We have some Go modules in the BCR but I assume they are in there when the Gazelle doesn't work perfectly and needs overriding with custom changes. Happy to close this.

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