We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
For security reasons "request" should be removed in the package.json. It is used in the following places:
fetch should be a good replacement as we are using node v18 as a minimum now.
fetch
CLI
The text was updated successfully, but these errors were encountered:
I'll take this!
Sorry, something went wrong.
Titanium CLI v7 requires Node.js 18 or newer, the SDK requires Node.js 16: https://github.com/tidev/titanium-sdk/blob/master/package.json#L172. We should use fetch() only if master is slated to be 13.0.0. If we plan to ship a 12.6.0, then we must use a user land package and I highly recommend undici.
fetch()
master
Bah, and wouldn't you know it, undici requires Node 18. Nevermind that.
@AbdullahFaqeir you are welcome to do the work, but we can only merge it if master becomes 13.0.0.
Eeeeh 🤗, I'll just do it for fun 😂, and we can all wait for the merge.
No branches or pull requests
I have searched and made sure there are no existing issues for the issue I am filing
Description
For security reasons "request" should be removed in the package.json. It is used in the following places:
fetch
should be a good replacement as we are using node v18 as a minimum now.Solution
Alternatives
Platforms
CLI
The text was updated successfully, but these errors were encountered: