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

fix: add proper version control to deps #2710

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

haikal-handamara-x150s
Copy link

@haikal-handamara-x150s haikal-handamara-x150s commented Feb 28, 2024

Summary

Package tmp release new version 0.2.2 which changes the minimum node version from >=8.17.0 to >=14 which caused any packages that dependent to tmp may broken. This commit lock the tmp version to safe version which would not break any application that depends to exceljs@4.4.0 but running in older NodeJS version (<14).

Test plan

Normal CI/CD test, but on NodeJS <14

Related to source code (for typings update)

Nothing

Package `tmp` release new version 0.2.2 which changes the minimum node version from `>=8.17.0` to `>=14` which caused any packages that dependent to `tmp` may broken
@pnappa
Copy link

pnappa commented Feb 29, 2024

I really hope you're not using such an old version of NodeJS in a production environment. Version 8 hasn't had any fixes for four years. https://endoflife.date/nodejs

Please update to a newer version (e.g. 18 or 20). You are at risk of security vulnerabilities.

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.

None yet

2 participants