-
Notifications
You must be signed in to change notification settings - Fork 3k
Closed
Labels
7.xIssues and PRs for version 7.xIssues and PRs for version 7.x8.xIssues and PRs for version 8.xIssues and PRs for version 8.x
Description
I think this is unfortunate, because we might want to export something else as observable, and it's a bit confusing for people at times I imagine:
import { observable } from 'rxjs';
console.log(observable); // symbolWe're currently exporting our copy of Symbol.observable as observable from the main module. I think we should rename it, remove it, or move it under something else.
Options:
- Export it as
observableSymbolorOBSERVABLE_SYMBOL - Don't export it at all, and advise people to use a package like
symbol-observable - Export it as a static property of
Observableor something. likeObservable.SYMBOL. (minimal overhead there, it's more of stylistic thing, I'm not saying I like it. Just an option) - ???? Something else.
In all cases we'd have to leave the observable export for the duration of 7.x, and simply deprecate it.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
7.xIssues and PRs for version 7.xIssues and PRs for version 7.x8.xIssues and PRs for version 8.xIssues and PRs for version 8.x