-
Notifications
You must be signed in to change notification settings - Fork 316
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
Publish @tus/client
#522
Comments
How does one do this? Using the |
Indeed! One could argue that a new major is not needed when you deprecate, so that's up to us |
Ok, so I think I would do it as the following:
Does that sounds the correct plan? I do not think a major release is necessary here. |
Sounds good! |
To be honest, I had second thoughts about this. There are a lot of examples, documentation, blog posts and resources covering tus-js-client, so changing the name would be cumbersome for new and existing users without providing much benefit for them. From my own experience, I feel how frustrating it is if the name of a dependency changes that is not frequently interacted with (frankly, I do think that most developers do not use tus-js-client very often, once it has been successfully integrated). Yes, I do see the benefit of using the @tus scope for tus-node-server, which consists of multiple packages. But for tus-js-client that is not (yet) the case, so I don't see it as useful for this project. I like the idea of having everything under one umbrella, but when comparing the advantages and downsides, I still think that a name change would cause unnecessary confusion for users. So I am now tending to keeping the package name as tus-js-client. |
I think it might still be worth it. For two reasons:
This is about a long-term strategy, to have official packages made with expertise signal that they are by having them under |
Now that tus-node-server is published under the
@tus
npm scope,tus-js-client
should follow along.We need to:
tus-js-client
which marks this npm package as deprecated. It will no longer receive updates besides security ones. This also forces people to look up the changelog and they'll find out they need to switch.@tus/client
and publish to npm.The text was updated successfully, but these errors were encountered: