-
Notifications
You must be signed in to change notification settings - Fork 7.5k
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
public, 'current' version tag: v7 or v8 #8112
Comments
Hey! We've detected some video files in a comment on this issue. If you'd like to permanently archive these videos and tie them to this project, a maintainer of the project can reply to this issue with the following commands:
|
👋 Thanks for opening your first issue here! 👋 If you're reporting a 🐞 bug, please make sure you include steps to reproduce it. We get a lot of issues on this repo, so please be patient and we will get back to you as soon as we can. |
The npm tags tend to be more up-to-date compared to the github tags. We don't tend to clear out the The DT types do work for Video.js v7 and older, though. But deprecating it for v8 sounds fine. Though, probably fix some things like #8109 before this is deprecated. |
@gkatsev thanks! subscribed to 8109, thanks! |
Closing this as it seems to be resolved and being tracked in another issue. Please re-open if needed. |
Hi,
not a code issue:
This is for maintainers of the repo. I'd like to create a PR against Definitely Typed repository to deprecated v7 of
@types/video.js
and to redirect types acquisition for@latest
version against current NPM registry. But I have a doubt, as the repository lists v8 recent releases asprerelease
, while NPM registry lists8.0.4
version as both latest and next versions.npm pack
seems to confirm v8 islatest
version, non-prerelease.https://www.npmjs.com/package/video.js?activeTab=versions
https://unpkg.com/browse/[email protected]/dist/types/video.d.ts
As the results, the v8 will be picked as the version suggested by automatic type acquisition feature in TS (VS/VSCode,etc), unless the entry in
package.json
suggests v7 (or older)Thanks!
The text was updated successfully, but these errors were encountered: