cd263663b3332ac10da6721dee8f52804639f19b,src/python/pants/option/arg_splitter.py,ArgSplitter,split_args,#ArgSplitter#Any#,37

Before Change

      self._unconsumed_args.pop()

    target = self._consume_target()
    while target:
      targets.append(target)
      target = self._consume_target()

    // We parse the word "help" as a scope, but it"s not a real one, so ignore it.
    scope_to_flags.pop("help", None)
    return scope_to_flags, targets

  def _consume_scope(self):

After Change

    while self._unconsumed_args:
      arg = self._unconsumed_args.pop()
      if arg.startswith(b"-"):
        // During migration we allow flags here, and assume they are in global scope.
        // TODO(benjy): Should we allow this even after migration?
        scope_to_flags[GLOBAL_SCOPE].append(arg)

Instances