Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update packages to latest versions #79

Merged
merged 2 commits into from
Feb 28, 2024
Merged

Conversation

KodrAus
Copy link
Member

@KodrAus KodrAus commented Feb 28, 2024

This PR just updates our packages to their most recent versions, resolving some vulnerabilities in transient dependencies. I've also removed the nyc dev dependency, which we didn't appear to be using and pulled in a lot of stuff.

@KodrAus
Copy link
Member Author

KodrAus commented Feb 28, 2024

Ok, looks like we can't update node-fetch without dropping support for Node versions below 18:

Error [ERR_REQUIRE_ESM]: require() of ES Module /home/runner/work/seq-logging/seq-logging/node_modules/node-fetch/src/index.js from /home/runner/work/seq-logging/seq-logging/index.js not supported.
Instead change the require of /home/runner/work/seq-logging/seq-logging/node_modules/node-fetch/src/index.js in /home/runner/work/seq-logging/seq-logging/index.js to a dynamic import() which is available in all CommonJS modules.

@nblumhardt nblumhardt merged commit 560e964 into datalust:dev Feb 28, 2024
13 checks passed
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