video.js icon indicating copy to clipboard operation
video.js copied to clipboard

Fix/replace global with videojs global compat

Open ronalduQualabs opened this issue 1 month ago • 2 comments

remove global and use 'videojs-global-compat' to fix https://github.com/videojs/video.js/issues/9103

ronalduQualabs avatar Nov 06 '25 20:11 ronalduQualabs

💖 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.

welcome[bot] avatar Nov 06 '25 20:11 welcome[bot]

Codecov Report

:white_check_mark: All modified and coverable lines are covered by tests. :white_check_mark: Project coverage is 84.35%. Comparing base (20f8d76) to head (faa6cde).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #9118      +/-   ##
==========================================
- Coverage   84.89%   84.35%   -0.54%     
==========================================
  Files         120      120              
  Lines        8154     8154              
  Branches     1964     1964              
==========================================
- Hits         6922     6878      -44     
- Misses       1232     1276      +44     

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

:rocket: New features to boost your workflow:
  • :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • :package: JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

codecov[bot] avatar Nov 06 '25 20:11 codecov[bot]