jellyfin / jellyfin-expo

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

Refactor fetch with timeout to separate utility #353

Closed thornbill closed 2 years ago

thornbill commented 2 years ago

Moves the fetch with timeout functionality to a dedicated utility function for future reuse

codecov[bot] commented 2 years ago

Codecov Report

Merging #353 (92d5413) into master (5606893) will increase coverage by 0.49%. The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #353      +/-   ##
==========================================
+ Coverage   48.26%   48.75%   +0.49%     
==========================================
  Files          39       40       +1     
  Lines         719      722       +3     
  Branches      143      143              
==========================================
+ Hits          347      352       +5     
+ Misses        335      333       -2     
  Partials       37       37              
Impacted Files Coverage Δ
utils/Fetch.js 100.00% <100.00%> (ø)
utils/ServerValidator.js 100.00% <100.00%> (+4.08%) :arrow_up:

: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
1.4% 1.4% Duplication