Skip to content

docs(types): document Player.error(null) for clearing MediaError#9174

Open
CyberVy wants to merge 1 commit intovideojs:mainfrom
CyberVy:null_error_type
Open

docs(types): document Player.error(null) for clearing MediaError#9174
CyberVy wants to merge 1 commit intovideojs:mainfrom
CyberVy:null_error_type

Conversation

@CyberVy
Copy link
Copy Markdown

@CyberVy CyberVy commented Apr 23, 2026

Updated documentation for MediaError handling to clarify parameters.

Description

player.error supports null as an input to clear the current MediaError, however, this is not revealed in the docs which can impact type build.

Updated documentation for MediaError handling to clarify parameters.
@CyberVy
Copy link
Copy Markdown
Author

CyberVy commented Apr 24, 2026

Could someone please review this PR, it'a just a simple fix. Thank you.

@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 24, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 84.32%. Comparing base (69ea0e0) to head (f08bd8b).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #9174   +/-   ##
=======================================
  Coverage   84.32%   84.32%           
=======================================
  Files         120      120           
  Lines        8153     8153           
  Branches     1964     1964           
=======================================
  Hits         6875     6875           
  Misses       1278     1278           

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

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants