Skip to content

Conversation

@teogeb
Copy link
Collaborator

@teogeb teogeb commented Jan 23, 2025

Resolves also this issues in Docker image build:

#19 0.931 npm warn EBADENGINE Unsupported engine {
#19 0.931 npm warn EBADENGINE   package: '[email protected]',
#19 0.931 npm warn EBADENGINE   required: { node: '20 || >=22' },
#19 0.931 npm warn EBADENGINE   current: { node: 'v18.20.6', npm: '10.8.2' }
#19 0.931 npm warn EBADENGINE }

teogeb added a commit that referenced this pull request Jan 23, 2025
Applied these API changes:
- `nodeInfo.version` field has been renamed to `applicationVersion`
- some type assertions removed as `NodeInfo` has stricter type
- `user` field has been renamed to `userId` in permission API
- stream partition count is queried via `getStreamPartitionCount()`
function
- `environment` config option is used instead of `CONFIG_TEST`
- `waitForCondition()` test utility has been renamed `until()`
- use `fetchPrivateKeyWithGas()` instead of hardcoded private key for
end-to-end test

The Docker build requires update to Node 20/22. This is done in a
separate PR: #28
@teogeb teogeb force-pushed the upgrade-to-node-22 branch from efa49c3 to e5765e5 Compare January 23, 2025 15:37
@teogeb teogeb force-pushed the upgrade-to-node-22 branch from e5765e5 to ce19505 Compare January 23, 2025 15:52
@teogeb teogeb changed the title deps: Bump NodeJS to v22 deps: [NET-1410] Bump NodeJS to v22 Jan 23, 2025
@linear
Copy link

linear bot commented Jan 23, 2025

@teogeb teogeb requested a review from jtakalai January 27, 2025 10:07
@teogeb teogeb merged commit 64dd598 into main Jan 27, 2025
3 checks passed
@teogeb teogeb deleted the upgrade-to-node-22 branch January 27, 2025 13:21
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.

3 participants