-
Notifications
You must be signed in to change notification settings - Fork 136
Conversation
Can one of the admins verify this patch? |
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.
Thanks for the fix.
README.md
Outdated
@@ -28,10 +28,10 @@ git_repository( | |||
|
|||
load( | |||
"@io_bazel_rules_docker//docker:docker.bzl", |
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.
We should load this from container:container.bzl
if we're going to use this name (which we should!).
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.
Well spotted. Thanks and fixed.
Update the docs for the docker_rules.
According to https://docs.bazel.build/versions/master/command-line-reference.html#run run inherits from build and therefore we can omit the entry for run.
@@ -203,7 +203,6 @@ the following: | |||
```shell | |||
$ cat .bazelrc | |||
build --workspace_status_command=./print-workspace-status.sh | |||
run --workspace_status_command=./print-workspace-status.sh |
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.
Why did you remove this?
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.
See commit message:
Simplify .bazelrc example. …
According to
https://docs.bazel.build/versions/master/command-line-reference.html#run
run inherits from build and therefore we can omit the entry for run.
Cool thanks. Travis only runs properly from PRs coming from branches on the main repo (b/c of secrets), and I'm just going to skip Bazel CI because this is README only. |
Update the docs for the docker_rules.