Reformat with buildifier --warnings=all (#138)

Fixes some dictionaries to put things in the more common orders. Buildifer
used to default to doing this check and reformatting which is why the
//conditions:default got moved up in these in the first place.
This commit is contained in:
Thomas Van Lenten 2019-04-02 11:48:08 -04:00 committed by GitHub
parent 084758ff75
commit 2b1c4610c3
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 5 additions and 5 deletions

View File

@ -39,8 +39,8 @@ def _ximpl(ctx):
return _common_impl(ctx, True) return _common_impl(ctx, True)
_ATTRS = { _ATTRS = {
"content": attr.string_list(mandatory = False, allow_empty = True),
"out": attr.output(mandatory = True), "out": attr.output(mandatory = True),
"content": attr.string_list(mandatory = False, allow_empty = True),
} }
_write_file = rule( _write_file = rule(

View File

@ -25,7 +25,7 @@ def _with_or_test(ctx):
# dictionary from a select(). # dictionary from a select().
# Test select()-compatible input syntax. # Test select()-compatible input syntax.
input_dict = {"//conditions:default": ":d1", ":foo": ":d1"} input_dict = {":foo": ":d1", "//conditions:default": ":d1"}
asserts.equals(env, input_dict, selects.with_or_dict(input_dict)) asserts.equals(env, input_dict, selects.with_or_dict(input_dict))
# Test OR syntax. # Test OR syntax.
@ -38,17 +38,17 @@ def _with_or_test(ctx):
# Test mixed syntax. # Test mixed syntax.
mixed_dict = { mixed_dict = {
"//conditions:default": ":d3",
(":bar", ":baz"): ":d2",
":foo": ":d1", ":foo": ":d1",
(":bar", ":baz"): ":d2",
"//conditions:default": ":d3",
} }
asserts.equals( asserts.equals(
env, env,
{ {
"//conditions:default": ":d3",
":bar": ":d2", ":bar": ":d2",
":baz": ":d2", ":baz": ":d2",
":foo": ":d1", ":foo": ":d1",
"//conditions:default": ":d3",
}, },
selects.with_or_dict(mixed_dict), selects.with_or_dict(mixed_dict),
) )