jellyfin / jellyfin-expo

Jellyfin Mobile for iOS
https://jellyfin.org
Mozilla Public License 2.0
304 stars 66 forks source link

Add experimental native audio player #339

Closed thornbill closed 2 years ago

thornbill commented 2 years ago

Remaining issues:

sonarcloud[bot] commented 2 years ago

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 1 Code Smell

No Coverage information No Coverage information
1.5% 1.5% Duplication

codecov[bot] commented 2 years ago

Codecov Report

Merging #339 (9826f8b) into master (4986e0b) will decrease coverage by 1.12%. The diff coverage is 33.33%.

@@            Coverage Diff             @@
##           master     #339      +/-   ##
==========================================
- Coverage   48.75%   47.63%   -1.13%     
==========================================
  Files          40       41       +1     
  Lines         722      781      +59     
  Branches      143      162      +19     
==========================================
+ Hits          352      372      +20     
- Misses        333      365      +32     
- Partials       37       44       +7     
Impacted Files Coverage Δ
components/NativeShellWebView.js 16.66% <0.00%> (-1.25%) :arrow_down:
components/VideoPlayer.js 25.39% <ø> (ø)
screens/HomeScreen.js 0.00% <0.00%> (ø)
screens/DevSettingsScreen.js 62.50% <25.00%> (-37.50%) :arrow_down:
components/AudioPlayer.js 34.14% <34.14%> (ø)
stores/MediaStore.js 100.00% <100.00%> (ø)
stores/SettingStore.js 100.00% <100.00%> (ø)
utils/StaticScriptLoader.js 100.00% <100.00%> (ø)

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

sonarcloud[bot] commented 2 years ago

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication