-
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.
Awesome! thanks for the PR.
@@ -136,12 +136,14 @@ def _common_impl(ctx): | |||
files = [ctx.executable.resolver] | |||
|
|||
cluster_arg = ctx.attr.cluster | |||
cluster_arg = ctx.expand_make_variables("cluster", cluster_arg, {}) |
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.
For the sake of completeness, can you add this for namespace
and image_chroot
as well? (any anything else I might be forgetting).
FYI the process for this (since Travis won't give secrets to PRs from other repos) is that once reviewed I'll merge this into a branch and open a second PR to make sure we don't break the integration tests. This is awesome, thanks again. |
Just added |
@bazel-io Test this please |
* implemented make vars expansion * added make vars expansion for namespace and image_chroot * Fix buildifier issues
This PR is to propose a fix to #77