Remove _kwmarker
python and starlark already have syntax for this. Test: Presubmits Change-Id: I9082c1cc62837840f091cd5856b1b4cc39064274
This commit is contained in:
parent
092c6b68b8
commit
b9b54396d4
1 changed files with 1 additions and 6 deletions
|
@ -80,23 +80,18 @@ _all_values_schema = {
|
||||||
},
|
},
|
||||||
}
|
}
|
||||||
|
|
||||||
def flag(name, partitions, default, _kwmarker = (), appends = False):
|
def flag(name, partitions, default, *, appends = False):
|
||||||
"""Declare a flag.
|
"""Declare a flag.
|
||||||
|
|
||||||
Args:
|
Args:
|
||||||
name: name of the flag
|
name: name of the flag
|
||||||
partitions: the partitions where this should be recorded.
|
partitions: the partitions where this should be recorded.
|
||||||
default: the default value of the flag.
|
default: the default value of the flag.
|
||||||
_kwmarker: Used to detect argument misuse.
|
|
||||||
appends: Whether new values should be append (not replace) the old.
|
appends: Whether new values should be append (not replace) the old.
|
||||||
|
|
||||||
Returns:
|
Returns:
|
||||||
A dictionary containing the flag declaration.
|
A dictionary containing the flag declaration.
|
||||||
"""
|
"""
|
||||||
|
|
||||||
# If specified, appends must be a keyword value.
|
|
||||||
if _kwmarker != ():
|
|
||||||
fail("Too many positional parameters")
|
|
||||||
if not partitions:
|
if not partitions:
|
||||||
fail("At least 1 partition is required")
|
fail("At least 1 partition is required")
|
||||||
if not name.startswith("RELEASE_"):
|
if not name.startswith("RELEASE_"):
|
||||||
|
|
Loading…
Reference in a new issue