project-slippi / slippi-js

Parse slp files and compute stats
GNU Lesser General Public License v3.0
147 stars 78 forks source link

Fix bad lint for fs.writeSync invocation in slpFile.ts #46

Closed jmlee337 closed 3 years ago

jmlee337 commented 3 years ago

whoever wrote the line originally mixed up the two method signatures for writeSync. The reason it worked to begin with is that the length argument is ignore if not a number.

add a test to ensure correctness, since non-trivial effort is required to test manually.