unjs / get-port-please

๐Ÿ”Œ Get an available open port
MIT License
255 stars 13 forks source link

fix: validate hostname and improve errors #59

Closed pi0 closed 1 year ago

pi0 commented 1 year ago

๐Ÿ”— Linked issue

โ“ Type of change

๐Ÿ“š Description

Resolves #58

New errors:

๐Ÿ“ Checklist

codecov[bot] commented 1 year ago

Codecov Report

Merging #59 (ad58820) into main (8d48ec6) will increase coverage by 1.75%. The diff coverage is 84.78%.

:exclamation: Current head ad58820 differs from pull request most recent head f025e58. Consider uploading reports for the commit f025e58 to get more accurate results

@@            Coverage Diff             @@
##             main      #59      +/-   ##
==========================================
+ Coverage   84.63%   86.39%   +1.75%     
==========================================
  Files           2        2              
  Lines         319      338      +19     
  Branches       47       55       +8     
==========================================
+ Hits          270      292      +22     
+ Misses         49       46       -3     
Files Changed Coverage ฮ”
src/index.ts 81.30% <84.78%> (+2.88%) :arrow_up: