previous CI setup wasn't actually running, it was throwing an uncaught rejection and somehow exiting with code 0 anyway
getting vitest to work on node 12 was miles away from working, so moving the node version tests to the test_tgz job, which is a simpler test that ensures the basics work.
Also adds some small fixes to the library itself to get it closer to working, but the glob dependency is the sticking point for node 12. 14, 16, 18 are now fine.
glob dependency will be replaced in #660
If getting node 12 to work proves impossible, we might need to just go to a v4 release but hopefully not.
github-actions[bot]
commented
6 months ago
previous CI setup wasn't actually running, it was throwing an uncaught rejection and somehow exiting with code 0 anyway
getting vitest to work on node 12 was miles away from working, so moving the node version tests to the
test_tgzjob, which is a simpler test that ensures the basics work.Also adds some small fixes to the library itself to get it closer to working, but the
globdependency is the sticking point for node 12. 14, 16, 18 are now fine.globdependency will be replaced in #660If getting node 12 to work proves impossible, we might need to just go to a v4 release but hopefully not.