Skip to content
This repository was archived by the owner on Nov 18, 2024. It is now read-only.

dart:cli documentation is missing #81

Closed
Hixie opened this issue Dec 21, 2020 · 13 comments
Closed

dart:cli documentation is missing #81

Hixie opened this issue Dec 21, 2020 · 13 comments
Assignees

Comments

@Hixie
Copy link

Hixie commented Dec 21, 2020

The https://api.dart.dev/ documentation is missing the "dart:cli" package.
It still exists, as demonstrated by running this program:

import 'dart:cli';
void main() => print(waitFor);

It used to be documented, for example, you can Google around and find this: https://api.dart.dev/stable/2.1.0/dart-cli/dart-cli-library.html

@franklinyow
Copy link
Contributor

CC: @kwalrath

@kwalrath kwalrath transferred this issue from dart-lang/sdk Jan 5, 2021
@kwalrath
Copy link

kwalrath commented Jan 5, 2021

Not sure whether this should be in the sdk repo or api.dart.dev repo... feel free to bounce it back. :)

@kwalrath
Copy link

kwalrath commented Jan 7, 2021

@devoncarew who'd be a good person to take this on?

@devoncarew
Copy link

This is by design; dart:cli is just one method, w/ a bunch of controversy about whether we should have shipped a one-method top-level core library.

cc @mit-mit

I believe this is WIA.

@Hixie
Copy link
Author

Hixie commented Jan 7, 2021

Having a secret method seems really weird. We should either decide we don't want this API and remove it, or we should document it, IMHO. Right now we're in a weird situation where the API exists and people ask questions about it but to answer their questions we have to point them to old API docs even though it still exists.

@mit-mit
Copy link
Contributor

mit-mit commented Jan 8, 2021

It's not secret, but it is experimental, hence it's not in the stable API docs.

I agree we should work towards either graduating it from being an experiment, or deprecate and then discontinue it.

@Hixie
Copy link
Author

Hixie commented Jan 8, 2021

Right now it looks like a fully supported feature we forgot to document (or worse, which we did document but then the documentation broke). If we want to describe it as an experiment we should document it but say "this is experimental".

@Hixie
Copy link
Author

Hixie commented Sep 26, 2021

I would like to escalate this. We should either document it or remove it but having undocumented features is IMHO not an acceptable middle ground.

@Hixie
Copy link
Author

Hixie commented Sep 26, 2021

cc @leafpetersen

@Hixie
Copy link
Author

Hixie commented Sep 26, 2021

(see also dart-lang/sdk#39390)

@mit-mit mit-mit self-assigned this Sep 27, 2021
@aam
Copy link

aam commented Sep 27, 2021

https://dart-review.googlesource.com/c/sdk/+/210125 landed some time ago that did 'deprecate the dart:cli library and waitFor method.'

@mit-mit
Copy link
Contributor

mit-mit commented Sep 28, 2021

copybara-service bot pushed a commit to dart-lang/sdk that referenced this issue Oct 27, 2021
Clarify this is supported, but deprecated, and may get removed
in the future should we find a suitable replacement.

Bug: dart-archive/api.dart.dev#81
Bug: #39390
Change-Id: I1304a0862986ae1b365a063ee75f0041341523f5
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/214620
Commit-Queue: Michael Thomsen <[email protected]>
Reviewed-by: Lasse R.H. Nielsen <[email protected]>
@mit-mit
Copy link
Contributor

mit-mit commented Sep 14, 2022

This was fixed a while back

@mit-mit mit-mit closed this as completed Sep 14, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

No branches or pull requests

6 participants