-
Notifications
You must be signed in to change notification settings - Fork 3k
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.
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
Specify deprecation notice in rxjs-es
package
#2218
Comments
Npm still supports the deprecate command, I believe? It would then warn the user when they install. Ben will have to do it I think. |
If that's supported, that's good 👍 |
👍 for npm deprecating this if it is indeed abandoned. I still see recommendations to use |
The official documentation still recommends installing |
@benlesh ^^^ they just need to be published |
@jayphelps I think what's in master would be in a bad state for publishing docs just now because of the extra docs around the new lettable operators. Once we get that fixed, I'll republish the docs. |
@benlesh could you checkout the commit before the stuff you don't want, then publish from there? |
@jayphelps .. I could... might be a bit of digging. |
Closing this. I don't think this is any longer an issue. |
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
RxJS version:
b.12
Code to reproduce:
https://www.npmjs.com/package/rxjs-es
Expected behavior:
Actual behavior:
Additional information:
Currently
rxjs-es
package is stay there with last available (b.12, https://www.npmjs.com/package/rxjs-es) , and there isn't any update further since newly published package won't include any ES2015 packages. NPM no longer allows delete package so anyone search npm packages can find, install unsupported package. While it is not available to remove packages, at least it is possible to specify given package's deprecated and not recommended to use - such asghooks
does recently (https://www.npmjs.com/package/ghooks), by publishing one dummy package with deprecation notice.Thoughts? /cc @Blesh, @jayphelps .
The text was updated successfully, but these errors were encountered: