feature: expose version from player.version()

[Svarozic]

Description

Expose version from player.version(), Issue #8538

Specific Changes proposed

Please list the specific changes involved in this pull request.

Requirements Checklist

• Feature implemented / Bug fixed
• If necessary, more likely in a feature request than a bug fix
  • Change has been verified in an actual browser (Chrome, Firefox, IE)
  • Unit Tests updated or fixed
  • Docs/guides updated
  • Example created (starter template on JSBin)
• Reviewed by Two Core Contributors

[welcome]

💖 Thanks for opening this pull request! 💖

Things that will help get your PR across the finish line:

• Run npm run lint -- --errors locally to catch formatting errors earlier.
• Include tests when adding/changing behavior.
• Include screenshots and animated GIFs whenever possible.

We get a lot of pull requests on this repo, so please be patient and we will get back to you as soon as we can.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (09c9bfb) 82.71% compared to head (dbdb1b9) 82.71%.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #8543   +/-   ##
=======================================
  Coverage   82.71%   82.71%           
=======================================
  Files         113      113           
  Lines        7612     7614    +2     
  Branches     1830     1830           
=======================================
+ Hits         6296     6298    +2     
  Misses       1316     1316           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[gkatsev]

Thanks for the PR!

[gkatsev]

I think rather than returning just the string, we should return an object with a video.js property (to match package name) for this version.

This way, we can extend this to add plugin and tech versions in here too, similar to https://github.com/videojs/http-streaming/blob/main/src/videojs-http-streaming.js#L1173-L1180

[Svarozic]

changed 3b10e8c

[amtins]

LGTM! Thanks again for your contribution 👍🏽

[gkatsev]

LGTM!

[welcome]

Congrats on merging your first pull request! 🎉🎉🎉

[Svarozic]

@gkatsev @amtins big thank you for your support in my created issue and this PR :)