PaulCombal commented 3 years ago

Hello, This pull request aims to address #105 . It was not heavily tested, and I had no linter. That means it's more a proof-of-concept than anything else, but with a little work, I'm sure it can reach your standards if you wanted to.

This code is functionnal on my side.

Maintainers are allowed to edit this PR.

Thank you for reviewing!

codecov[bot] commented 3 years ago

Codecov Report

Merging #106 (0a8108f) into master (0a8108f) will not change coverage. The diff coverage is n/a.

@@            Coverage Diff            @@
##            master      #106   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            3         3           
  Lines          255       255           
=========================================
  Hits           255       255

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 0a8108f...00135f3. Read the comment docs.

lgtm-com[bot] commented 3 years ago

This pull request introduces 1 alert when merging 00135f338d102452df9085039f9aa81fe3d01e3f into 0a8108f913f450fe74ab41c3d88335e91ea16def - view on LGTM.com

new alerts:

1 for Unused variable, import, function or class

TimeForANinja / node-ytpl

Added channelPlaylists concept #106

Codecov Report