From b9b54396d4a71f72a04ac631368890cc71ad8bba Mon Sep 17 00:00:00 2001 From: Cole Faust Date: Thu, 9 Nov 2023 12:40:38 -0800 Subject: [PATCH] Remove _kwmarker python and starlark already have syntax for this. Test: Presubmits Change-Id: I9082c1cc62837840f091cd5856b1b4cc39064274 --- core/release_config.scl | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/core/release_config.scl b/core/release_config.scl index 101f119709..662d15507a 100644 --- a/core/release_config.scl +++ b/core/release_config.scl @@ -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. Args: name: name of the flag partitions: the partitions where this should be recorded. default: the default value of the flag. - _kwmarker: Used to detect argument misuse. appends: Whether new values should be append (not replace) the old. Returns: A dictionary containing the flag declaration. """ - - # If specified, appends must be a keyword value. - if _kwmarker != (): - fail("Too many positional parameters") if not partitions: fail("At least 1 partition is required") if not name.startswith("RELEASE_"):