Remove -c, --[no-]checked from CLI usage text

[chalin]

From #1807 (comment):

The option --[no-]checked should be removed/replaced.

> pub run -h
Run an executable from a package.

Usage: pub run <executable> [args...]
-h, --help            Print this usage information.
-c, --[no-]checked    Enable runtime type checks and assertions.

Run "pub help" to see global options.

cc @kevmoo @kwalrath

[kevmoo]

Maybe just add --[no-]enable-asserts and hide --checked?

[kevmoo]

Sorry. This is for pub client, not pkg:test – sorry for the fire drill @grouma

[natebosch]

In Dart2 strong mode, the checked parameter only controls asserts, but not type checks.

https://api.dartlang.org/dev/2.0.0-dev.63.0/dart-isolate/Isolate/spawnUri.html

--enable-asserts would have the same implementation as --checked today. I'd be fine adding an alias for this and hiding the original.

[natebosch]

@kevmoo - what should the single character flag be?

[kevmoo]

e? a?

[chalin]

--enable-asserts would have the same implementation as --checked

In the context of Dart 2, since the old and the new flags have the same effect, how about this: renaming the new flag to --check-asserts and reusing -c as a shorthand.

Of course --no-checked and --checked would be dropped.

[chalin]

Any update on this?

[kevmoo]

@jakemac53 @natebosch ?

[natebosch]

Fixed with 9f00679