Skip to content

QSTH-937: Upgrade the dependencies and pin versions#13

Open
mehmettokgoz wants to merge 4 commits intomasterfrom
QSTH-937
Open

QSTH-937: Upgrade the dependencies and pin versions#13
mehmettokgoz wants to merge 4 commits intomasterfrom
QSTH-937

Conversation

@mehmettokgoz
Copy link
Copy Markdown
Collaborator

No description provided.

@mehmettokgoz mehmettokgoz requested a review from CahidArda April 1, 2026 10:27
@mehmettokgoz mehmettokgoz self-assigned this Apr 1, 2026
@linear
Copy link
Copy Markdown

linear bot commented Apr 1, 2026

Copy link
Copy Markdown

@ytkimirti ytkimirti left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

node-fetch@3.3.2 is ESM-only, but this project compiles to CommonJS ("module": "commonjs" in tsconfig.json). The postinstall script runs node install.js, so this will fail at runtime with:

Error [ERR_REQUIRE_ESM]: require() of ES Module node-fetch not supported

node-fetch v3 is ESM-only which breaks the CJS build. Use Node's
built-in fetch with Readable.fromWeb() instead.
CI overwrites the version from the git tag at release time,
so the value in package.json is irrelevant.
install.js is a build artifact generated by tsc from install.ts.
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