-
Notifications
You must be signed in to change notification settings - Fork 171
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
should we delete cujojs/rest? #375
Comments
Another option could be to move it to attic?
|
Another option could be to move it to attic?
I don't think it is significant enough code to do that. Usually we only
attic things that are repo scale whereas this rest interceptor is quite
small. source history will still have it though
|
I am up for removing it then.
|
Ping @jpiechowka. Are you still using it? |
I added the 1.0 flag as it is very likely we are removing cuojs for the new API |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
It looks like it is abandoned..
https://www.npmjs.com/package/rest (3 years old)
https://github.com/cujojs/rest (archived repo)
cc @jpiechowka @pchiwan @pbadenski @muymoo @openzipkin/zipkin-js-champions
The text was updated successfully, but these errors were encountered: