ArgsResult should have a way to determine if the user actual set the value about args HOT 8 CLOSED

<img src="https://avatars.githubusercontent.com/u/17034?v=3" align="left" width="48" height="48"hspace="10"> Comment by kevmoo

Added Area-Library, Library-Args, Triaged labels.

from args.

<img src="https://avatars.githubusercontent.com/u/4865287?v=3" align="left" width="48" height="48"hspace="10"> Comment by lrhn

Removed Area-Library label.
Added Area-Pkg label.

from args.

<img src="https://avatars.githubusercontent.com/u/444270?v=3" align="left" width="48" height="48"hspace="10"> Comment by seaneagan

Interesting. Dart used to have an equivalent feature for function arguments, until it was removed by popular demand (issue dart-lang/sdk#9097).

If the default value is calculated like that, then does it make sense to have an explicit default value? Seems like it might make more sense to just explain the default calculation in the help:, and leave null as the default value, in which case you can tell whether a value was specified since there is no way to pass null from the command line. However for flags, defaultsTo defaults to false, not null, so would need to do defaultsTo: null for that, haven't tested if that works.

from args.

This comment was originally written by [email protected]

I like having default values for a nice out of the box experience. I ended creating a settings object that takes a map as a source and provides the default values if they're not in the map. Then I merge Platform.environment and ArgResults[] into the map provided to the settings object. There's a bit of friction as ArgResults doesn't implement Map or provide an accessor to a Map view, so I have to loop through the options rather than just using Map.addAll. Any thoughts on why ArgResults doesn't implement map or provide an accessor to a Map view? I think having ArgResults implement Map or provide an accessor to a Map view might make merging configuration sources cleaner.

from args.

<img src="https://avatars.githubusercontent.com/u/3276024?v=3" align="left" width="48" height="48"hspace="10"> Comment by anders-sandholm

Removed Library-Args label.
Added Pkg-Args label.

from args.

<img src="https://avatars.githubusercontent.com/u/46275?v=3" align="left" width="48" height="48"hspace="10"> Comment by munificent

I like having default values for a nice out of the box experience.

Agreed. In particular, default values are shown in the usage information.

Any thoughts on why ArgResults doesn't implement map or provide an accessor to a Map view?

Args is super old so I think the map type was still in flux when I first wrote it. I'll follow up on the other bug for this.

Set owner to @munificent.
Removed Type-Defect label.
Added Type-Enhancement, Started labels.

from args.

<img src="https://avatars.githubusercontent.com/u/46275?v=3" align="left" width="48" height="48"hspace="10"> Comment by munificent

I just pushed arg 0.12.0, which adds a .wasParsed() method to ArgResults.

Added Fixed label.

from args.

This comment was originally written by [email protected]

Excellent. Thank you!

from args.