Skip to content
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

chore: update folder structure in the playback package #48

Merged
merged 1 commit into from
Jan 15, 2025

Conversation

dzianis-dashkevich
Copy link
Collaborator

@dzianis-dashkevich dzianis-dashkevich commented Jan 15, 2025

Description

Updated folder structure in the playback package. Renamed player files

Specific Changes proposed

N/A

Requirements Checklist

  • Feature implemented / Bug fixed
  • Change has been verified in an actual browser (Chrome, Firefox, Safari, Edge) (if applicable)
  • Unit Tests updated or fixed (if applicable)
  • Docs/guides updated (if applicable)

Copy link

netlify bot commented Jan 15, 2025

Deploy Preview for videojsdev ready!

Name Link
🔨 Latest commit 2b0e1b6
🔍 Latest deploy log https://app.netlify.com/sites/videojsdev/deploys/6788448867fbc800086fcb44
😎 Deploy Preview https://deploy-preview-48--videojsdev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

size-limit report 📦

Path Size Loading time (3g) Running time (snapdragon) Total time
dash-parser/dist/cjs/index.min.js 3.65 KB (0%) 73 ms (0%) 52 ms (-61.09% 🔽) 125 ms
dash-parser/dist/es/index.min.js 2.67 KB (0%) 54 ms (0%) 15 ms (+9.84% 🔺) 68 ms
dash-parser/dist/iife/index.min.js 3.65 KB (0%) 74 ms (0%) 80 ms (-59.85% 🔽) 153 ms
hls-parser/dist/cjs/index.min.js 5.81 KB (0%) 117 ms (0%) 148 ms (+33.75% 🔺) 264 ms
hls-parser/dist/es/index.min.js 3.37 KB (0%) 68 ms (0%) 25 ms (-59.29% 🔽) 92 ms
hls-parser/dist/iife/index.min.js 5.79 KB (0%) 116 ms (0%) 92 ms (+215.87% 🔺) 208 ms
playback/dist/main-only/core/cjs/index.min.js 0 B (-100% 🔽) 0 ms (-100% 🔽) 0 ms (-100% 🔽) 0 ms
playback/dist/main-only/core/es/index.min.js 0 B (-100% 🔽) 0 ms (-100% 🔽) 0 ms (-100% 🔽) 0 ms
playback/dist/main-only/core/iife/index.min.js 0 B (-100% 🔽) 0 ms (-100% 🔽) 0 ms (-100% 🔽) 0 ms
playback/dist/main-with-worker/core/cjs/index.min.js 0 B (-100% 🔽) 0 ms (-100% 🔽) 0 ms (-100% 🔽) 0 ms
playback/dist/main-with-worker/core/es/index.min.js 0 B (-100% 🔽) 0 ms (-100% 🔽) 0 ms (-100% 🔽) 0 ms
playback/dist/main-with-worker/core/iife/index.min.js 0 B (-100% 🔽) 0 ms (-100% 🔽) 0 ms (-100% 🔽) 0 ms
env-capabilities/dist/cjs/index.min.js 1.61 KB (0%) 33 ms (0%) 20 ms (+4.27% 🔺) 52 ms
env-capabilities/dist/es/index.min.js 0 B (+100% 🔺) 0 ms (+100% 🔺) 2 ms (-6.39% 🔽) 2 ms
env-capabilities/dist/iife/index.min.js 1.62 KB (0%) 33 ms (0%) 8 ms (-13.6% 🔽) 41 ms
playback/dist/player/core/cjs/index.min.js 5.95 KB (+100% 🔺) 119 ms (+100% 🔺) 257 ms (+100% 🔺) 376 ms
playback/dist/player/core/es/index.min.js 3.42 KB (+100% 🔺) 69 ms (+100% 🔺) 119 ms (+100% 🔺) 188 ms
playback/dist/player/core/iife/index.min.js 5.95 KB (+100% 🔺) 120 ms (+100% 🔺) 256 ms (+100% 🔺) 375 ms
playback/dist/player-with-worker/core/cjs/index.min.js 8.23 KB (+100% 🔺) 165 ms (+100% 🔺) 293 ms (+100% 🔺) 458 ms
playback/dist/player-with-worker/core/es/index.min.js 6.25 KB (+100% 🔺) 125 ms (+100% 🔺) 304 ms (+100% 🔺) 429 ms
playback/dist/player-with-worker/core/iife/index.min.js 8.23 KB (+100% 🔺) 165 ms (+100% 🔺) 280 ms (+100% 🔺) 445 ms

@dzianis-dashkevich dzianis-dashkevich merged commit cef7ecb into main Jan 15, 2025
9 checks passed
@dzianis-dashkevich dzianis-dashkevich deleted the rename-and-move-files branch January 15, 2025 23:34
Copy link

codecov bot commented Jan 15, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 63.89%. Comparing base (a86e68e) to head (2b0e1b6).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #48   +/-   ##
=======================================
  Coverage   63.89%   63.89%           
=======================================
  Files         114      114           
  Lines        4833     4833           
  Branches      633      633           
=======================================
  Hits         3088     3088           
  Misses       1737     1737           
  Partials        8        8           
Flag Coverage Δ
dash-parser 63.89% <100.00%> (ø)
env-capabilities 63.89% <100.00%> (ø)
hls-parser 63.89% <100.00%> (ø)
playback 63.89% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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

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.

2 participants