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 dependencies #1849

Merged
merged 7 commits into from
Dec 14, 2022
Merged

Update dependencies #1849

merged 7 commits into from
Dec 14, 2022

Conversation

lfdebrux
Copy link
Member

@lfdebrux lfdebrux commented Dec 14, 2022

Bumps a bunch of dependencies to make npm audit a bit happier.

@lfdebrux lfdebrux changed the title Bump dependencies Update dependencies Dec 14, 2022
We used to use fancy-log in our logging, however we removed the last
usage in commit 6b00c53. As this package is no longer used anywhere in
our code, we can remove it.
In PR #667 we added `acorn` as a direct dependency to fix complaints
from npm about peer dependencies, due to a bug in npm [[1]]. That bug
now appears to have been fixed; removing `acorn` from our `package.json`
does not result in any peer dependency warnings. Also, as we now ensure
that users do not install dev dependencies, we can be confident that
they will not need `acorn`, as it is only required by dev dependencies.

[1]: #666
@lfdebrux lfdebrux merged commit bef51cd into main Dec 14, 2022
@lfdebrux lfdebrux deleted the ldeb-bump-dependencies branch December 14, 2022 14:07
@lfdebrux lfdebrux added this to the 13.1.0 milestone Dec 21, 2022
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