Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

doc: remove "idiomatic choice" from queueMicrotask #23885

Closed
wants to merge 2 commits into from
Closed

doc: remove "idiomatic choice" from queueMicrotask #23885

wants to merge 2 commits into from

Commits on Oct 29, 2018

  1. doc: remove "idiomatic choice" from queueMicrotask 

    It can't be idiomatic if it's not in general use and therefore hasn't
been picked up by users. It's not even in browsers yet.

"Idiomatic" use is an emergent property that comes from observed use
and this feature is so new (to browsers and Node) that it can't
possibly be. In general I don't think it's the place of the Node API
docs to observe what emerges as idiomatic Node.js.

It also can't be a recommended feature (if that was the intent of the
language) because it's marked experimental. For now, it's just a
feature, nothing more. Recommendations and/or observations about it
being 'idiomatic' can come later.
    @rvagg
    rvagg committed Oct 29, 2018
    Configuration menu
    Copy the full SHA
    385510b View commit details
    Browse the repository at this point in the history

  2. fixup! doc: remove "idiomatic choice" from queueMicrotask

    @rvagg
    rvagg committed Oct 29, 2018
    Configuration menu
    Copy the full SHA
    a6da5fd View commit details
    Browse the repository at this point in the history