Skip to content

Conversation

boris-petrov
Copy link

@boris-petrov boris-petrov commented Jul 5, 2023

cc @rainfore

P.S. Please release this as a minor version, 0.8.1, as there are no breaking changes.

@andersk
Copy link

andersk commented Nov 7, 2024

This PR does not work.

$ npm run test

> @vusion/webfonts-generator@0.8.0 test
> mocha tests


Error [ERR_REQUIRE_ESM]: require() of ES Module /home/anders/js/webfonts-generator/node_modules/read-chunk/index.js from /home/anders/js/webfonts-generator/tests/test.js not supported.
Instead change the require of index.js in /home/anders/js/webfonts-generator/tests/test.js to a dynamic import() which is available in all CommonJS modules.
    at Object.<anonymous> (/home/anders/js/webfonts-generator/tests/test.js:8:17)
    at async formattedImport (/home/anders/js/webfonts-generator/node_modules/mocha/lib/nodejs/esm-utils.js:9:14)
    at async exports.requireOrImport (/home/anders/js/webfonts-generator/node_modules/mocha/lib/nodejs/esm-utils.js:42:28)
    at async exports.loadFilesAsync (/home/anders/js/webfonts-generator/node_modules/mocha/lib/nodejs/esm-utils.js:100:20)
    at async singleRun (/home/anders/js/webfonts-generator/node_modules/mocha/lib/cli/run-helpers.js:125:3)
    at async exports.handler (/home/anders/js/webfonts-generator/node_modules/mocha/lib/cli/run.js:370:5)

I’ve opened a replacement:

@andersk andersk mentioned this pull request Nov 7, 2024
@boris-petrov
Copy link
Author

Perhaps, yes, I haven't been using this package for a while now. I'll close this as the other PR is better.

@boris-petrov boris-petrov deleted the update-packages branch November 8, 2024 09:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants