unjs / get-port-please

๐Ÿ”Œ Get an available open port
MIT License
245 stars 12 forks source link

chore: run tests on windows #60

Closed pi0 closed 1 year ago

pi0 commented 1 year ago

๐Ÿ”— Linked issue

โ“ Type of change

๐Ÿ“š Description

cost to well test all platforms

๐Ÿ“ Checklist

codecov[bot] commented 1 year ago

Codecov Report

Merging #60 (f7cdf6c) into main (cf4317c) will not change coverage. Report is 1 commits behind head on main. The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #60   +/-   ##
=======================================
  Coverage   85.13%   85.13%           
=======================================
  Files           5        5           
  Lines         370      370           
  Branches       48       48           
=======================================
  Hits          315      315           
  Misses         55       55           
pi0 commented 1 year ago

(Merging with now known windows issue!)

pi0 commented 1 year ago

Update: Locally testing, windows listens to one IPV6 host even when input host is 192.168.1.199 ! https://github.com/unjs/get-port-please/commit/e8b92ac16e28310423cc14c1bbb58b5e20836537 :}