-
Notifications
You must be signed in to change notification settings - Fork 44
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
lru-cache
dependency breaks the declared Node.js compatibility
#204
Comments
Suggesting to use |
I think I’ll probably drop support for v18 myself. I feel like supporting only active LTS versions is fine for a CI tool. |
Node 18.18+ is LTS (maintenance LTS) at the moment, @andrewbranch |
Dear @andrewbranch , you're making a very important tool for the community and I believe you can drop Node 18 in May 2025 (EOL is 2025-04-30), but at the moment it needs a little care: please review my PR, it works fine with previous version of |
The workaround for "resolutions": {
"@arethetypeswrong/**/lru-cache": "^10.4.3"
}, for |
Adding this line:
arethetypeswrong.github.io/packages/core/package.json
Line 56 in 8632156
Broke the Node.js compatibility of the package.
Commit: 33abbdb
The package Node.js compatibility is declared as
>= 18
, but that dependency is not compatible with v18.CC @andrewbranch
The text was updated successfully, but these errors were encountered: