-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
[credentialhelper] Update flag doc to point to more convenient usage instructions #21382
Conversation
…instructions Arguably, we should have a dedicated page for this on `bazel.build`, but until we have that, this post seems like a better source for users than the original design doc.
@bazel-io fork 7.1.0 |
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.
While the blog post clarifies how to use a credential helper, it doesn't describe how to write one. But I agree that the proposal isn't the right artifact to link to. How about we also link to https://github.com/EngFlow/credential-helper-spec/blob/main/spec.md?
SG, done. PTAL |
…instructions Arguably, we should have a dedicated page for this on `bazel.build`, but until we have that, this post seems like a better source for users than the original design doc. Closes bazelbuild#21382. PiperOrigin-RevId: 608747673 Change-Id: Iaca0dc8364a6d622f685bdd53eea54068c4ea94c
…t usage instructions (#21441) Arguably, we should have a dedicated page for this on `bazel.build`, but until we have that, this post seems like a better source for users than the original design doc. Closes #21382. Commit f1b02a2 PiperOrigin-RevId: 608747673 Change-Id: Iaca0dc8364a6d622f685bdd53eea54068c4ea94c Co-authored-by: Yannic <yannic@engflow.com>
The changes in this PR have been included in Bazel 7.1.0 RC1. Please test out the release candidate and report any issues as soon as possible. If you're using Bazelisk, you can point to the latest RC by setting USE_BAZEL_VERSION=last_rc. |
Arguably, we should have a dedicated page for this on
bazel.build
, but until we have that, this post seems like a better source for users than the original design doc.