Quick review (<5min). (Do not be fooled by the change numbers. It's all in package-lock.json!)
I encountered a problem importing some zarrita functions while writing unit tests for #78. Updating the test packages, while a good thing to do for maintenance, seemed to break everything even more. The solution to both problems was to convert this project to explicitly use ES modules (which provide the import Foo from "bar"; syntax) rather than CommonJS (where imports look like const Foo = require("bar");). Shout out to @toloudis for patiently wading with me through all these config and command line option changes.
While I was messing with config files, I also added format on save.
Quick review (<5min). (Do not be fooled by the change numbers. It's all in package-lock.json!)
I encountered a problem importing some zarrita functions while writing unit tests for #78. Updating the test packages, while a good thing to do for maintenance, seemed to break everything even more. The solution to both problems was to convert this project to explicitly use ES modules (which provide the
import Foo from "bar";
syntax) rather than CommonJS (where imports look likeconst Foo = require("bar");
). Shout out to @toloudis for patiently wading with me through all these config and command line option changes.While I was messing with config files, I also added format on save.