-
Notifications
You must be signed in to change notification settings - Fork 4
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
Upgrade to bazel 7.2.1 #60
Conversation
c943ccd
to
51f74cc
Compare
51f74cc
to
f4a0c5c
Compare
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.
Looks good for Language Tools!
@@ -13,7 +13,7 @@ manifests, image indexes (multi-arch images) and custom artifacts | |||
|
|||
In addition to Bazel rules, we offer many helpers for interacting with OCI | |||
artifacts under the `go/pkg` directory and a CLI tool for creating new OCI | |||
artifacts. You can add the Go library as a dependency by running `go get github.com/DataDog/rules_oci/go@latest`. |
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.
👏 praise: Thanks for making this change!
bazel_dep(name = "gazelle", version = "0.36.0") | ||
bazel_dep(name = "rules_go", version = "0.47.1") | ||
bazel_dep(name = "rules_pkg", version = "0.10.1") | ||
bazel_dep(name = "stardoc", version = "0.6.2") |
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.
👏 praise: I think this bump to stardoc
fixes this issue we had with some of our documentation.
This updates the repo to using bazel 7.2.1, and also updates the go dependencies. In addition, it removes some features that are not in use at DataDog: