TimeForANinja / node-ytpl

Anonymous YouTube playlist resolver.
MIT License
202 stars 48 forks source link

Update main.js — Change for the description #114

Closed Asgarrrr closed 3 years ago

Asgarrrr commented 3 years ago

info.description is an object, so the condition will always be false...

codecov[bot] commented 3 years ago

Codecov Report

Merging #114 (49cacae) into master (a04a718) will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##            master      #114   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            3         3           
  Lines          259       259           
=========================================
  Hits           259       259           
Impacted Files Coverage Δ
lib/main.js 100.00% <ø> (ø)

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 a04a718...49cacae. Read the comment docs.

TimeForANinja commented 3 years ago

😉 thanks again for the PR added some tests so that this doesn't happen again should be in the next release in a bit