perf: fix for cluster aggregation performance #26
SimenBAnnotations
120 errors and 25 warnings
|
|
|
|
|
Test on Node.js v16.x and OS ubuntu-latest:
lib/cluster.js#L190
Replace `(err)` with `err`
|
|
Test on Node.js v16.x and OS ubuntu-latest:
lib/cluster.js#L191
Replace `(err)·console.error(`Error·deleting·file·${message.filename}:`,·err)` with `·(err)⏎↹↹↹↹↹↹↹↹console.error(`Error·deleting·file·${message.filename}:`,·err);`
|
|
|
|
Test on Node.js v16.x and OS ubuntu-latest:
lib/cluster.js#L215
'i' is defined but never used
|
|
Test on Node.js v16.x and OS ubuntu-latest:
lib/cluster.js#L219
Replace `"hash"` with `'hash'`
|
|
Test on Node.js v16.x and OS ubuntu-latest:
lib/cluster.js#L219
["hash"] is better written in dot notation
|
|
Test on Node.js v16.x and OS ubuntu-latest:
lib/cluster.js#L223
Replace `os.tmpdir(),·`metrics-${process.pid}-${message.requestId}.json`` with `⏎↹↹↹↹↹↹↹os.tmpdir(),⏎↹↹↹↹↹↹↹`metrics-${process.pid}-${message.requestId}.json`,⏎↹↹↹↹↹↹`
|
|
Test on Node.js v16.x and OS ubuntu-latest:
lib/cluster.js#L224
Replace `(err)` with `err`
|
|
|
|
|
|
Test on Node.js v20.x and OS ubuntu-latest:
lib/cluster.js#L190
Replace `(err)` with `err`
|
|
Test on Node.js v20.x and OS ubuntu-latest:
lib/cluster.js#L191
Replace `(err)·console.error(`Error·deleting·file·${message.filename}:`,·err)` with `·(err)⏎↹↹↹↹↹↹↹↹console.error(`Error·deleting·file·${message.filename}:`,·err);`
|
|
|
|
Test on Node.js v20.x and OS ubuntu-latest:
lib/cluster.js#L215
'i' is defined but never used
|
|
Test on Node.js v20.x and OS ubuntu-latest:
lib/cluster.js#L219
Replace `"hash"` with `'hash'`
|
|
Test on Node.js v20.x and OS ubuntu-latest:
lib/cluster.js#L219
["hash"] is better written in dot notation
|
|
Test on Node.js v20.x and OS ubuntu-latest:
lib/cluster.js#L223
Replace `os.tmpdir(),·`metrics-${process.pid}-${message.requestId}.json`` with `⏎↹↹↹↹↹↹↹os.tmpdir(),⏎↹↹↹↹↹↹↹`metrics-${process.pid}-${message.requestId}.json`,⏎↹↹↹↹↹↹`
|
|
Test on Node.js v20.x and OS ubuntu-latest:
lib/cluster.js#L224
Replace `(err)` with `err`
|
|
|
|
|
|
Test on Node.js v16.x and OS macOS-latest:
lib/cluster.js#L190
Replace `(err)` with `err`
|
|
Test on Node.js v16.x and OS macOS-latest:
lib/cluster.js#L191
Replace `(err)·console.error(`Error·deleting·file·${message.filename}:`,·err)` with `·(err)⏎↹↹↹↹↹↹↹↹console.error(`Error·deleting·file·${message.filename}:`,·err);`
|
|
|
|
Test on Node.js v16.x and OS macOS-latest:
lib/cluster.js#L215
'i' is defined but never used
|
|
Test on Node.js v16.x and OS macOS-latest:
lib/cluster.js#L219
Replace `"hash"` with `'hash'`
|
|
Test on Node.js v16.x and OS macOS-latest:
lib/cluster.js#L219
["hash"] is better written in dot notation
|
|
Test on Node.js v16.x and OS macOS-latest:
lib/cluster.js#L223
Replace `os.tmpdir(),·`metrics-${process.pid}-${message.requestId}.json`` with `⏎↹↹↹↹↹↹↹os.tmpdir(),⏎↹↹↹↹↹↹↹`metrics-${process.pid}-${message.requestId}.json`,⏎↹↹↹↹↹↹`
|
|
Test on Node.js v16.x and OS macOS-latest:
lib/cluster.js#L224
Replace `(err)` with `err`
|
|
|
|
|
|
Test on Node.js v18.x and OS ubuntu-latest:
lib/cluster.js#L190
Replace `(err)` with `err`
|
|
Test on Node.js v18.x and OS ubuntu-latest:
lib/cluster.js#L191
Replace `(err)·console.error(`Error·deleting·file·${message.filename}:`,·err)` with `·(err)⏎↹↹↹↹↹↹↹↹console.error(`Error·deleting·file·${message.filename}:`,·err);`
|
|
|
|
Test on Node.js v18.x and OS ubuntu-latest:
lib/cluster.js#L215
'i' is defined but never used
|
|
Test on Node.js v18.x and OS ubuntu-latest:
lib/cluster.js#L219
Replace `"hash"` with `'hash'`
|
|
Test on Node.js v18.x and OS ubuntu-latest:
lib/cluster.js#L219
["hash"] is better written in dot notation
|
|
Test on Node.js v18.x and OS ubuntu-latest:
lib/cluster.js#L223
Replace `os.tmpdir(),·`metrics-${process.pid}-${message.requestId}.json`` with `⏎↹↹↹↹↹↹↹os.tmpdir(),⏎↹↹↹↹↹↹↹`metrics-${process.pid}-${message.requestId}.json`,⏎↹↹↹↹↹↹`
|
|
Test on Node.js v18.x and OS ubuntu-latest:
lib/cluster.js#L224
Replace `(err)` with `err`
|
|
|
|
|
|
Test on Node.js v21.x and OS ubuntu-latest:
lib/cluster.js#L190
Replace `(err)` with `err`
|
|
Test on Node.js v21.x and OS ubuntu-latest:
lib/cluster.js#L191
Replace `(err)·console.error(`Error·deleting·file·${message.filename}:`,·err)` with `·(err)⏎↹↹↹↹↹↹↹↹console.error(`Error·deleting·file·${message.filename}:`,·err);`
|
|
|
|
Test on Node.js v21.x and OS ubuntu-latest:
lib/cluster.js#L215
'i' is defined but never used
|
|
Test on Node.js v21.x and OS ubuntu-latest:
lib/cluster.js#L219
Replace `"hash"` with `'hash'`
|
|
Test on Node.js v21.x and OS ubuntu-latest:
lib/cluster.js#L219
["hash"] is better written in dot notation
|
|
Test on Node.js v21.x and OS ubuntu-latest:
lib/cluster.js#L223
Replace `os.tmpdir(),·`metrics-${process.pid}-${message.requestId}.json`` with `⏎↹↹↹↹↹↹↹os.tmpdir(),⏎↹↹↹↹↹↹↹`metrics-${process.pid}-${message.requestId}.json`,⏎↹↹↹↹↹↹`
|
|
Test on Node.js v21.x and OS ubuntu-latest:
lib/cluster.js#L224
Replace `(err)` with `err`
|
|
|
|
|
|
Test on Node.js v20.x and OS macOS-latest:
lib/cluster.js#L190
Replace `(err)` with `err`
|
|
Test on Node.js v20.x and OS macOS-latest:
lib/cluster.js#L191
Replace `(err)·console.error(`Error·deleting·file·${message.filename}:`,·err)` with `·(err)⏎↹↹↹↹↹↹↹↹console.error(`Error·deleting·file·${message.filename}:`,·err);`
|
|
|
|
Test on Node.js v20.x and OS macOS-latest:
lib/cluster.js#L215
'i' is defined but never used
|
|
Test on Node.js v20.x and OS macOS-latest:
lib/cluster.js#L219
Replace `"hash"` with `'hash'`
|
|
Test on Node.js v20.x and OS macOS-latest:
lib/cluster.js#L219
["hash"] is better written in dot notation
|
|
Test on Node.js v20.x and OS macOS-latest:
lib/cluster.js#L223
Replace `os.tmpdir(),·`metrics-${process.pid}-${message.requestId}.json`` with `⏎↹↹↹↹↹↹↹os.tmpdir(),⏎↹↹↹↹↹↹↹`metrics-${process.pid}-${message.requestId}.json`,⏎↹↹↹↹↹↹`
|
|
Test on Node.js v20.x and OS macOS-latest:
lib/cluster.js#L224
Replace `(err)` with `err`
|
|
|
|
|
|
Test on Node.js v18.x and OS macOS-latest:
lib/cluster.js#L190
Replace `(err)` with `err`
|
|
Test on Node.js v18.x and OS macOS-latest:
lib/cluster.js#L191
Replace `(err)·console.error(`Error·deleting·file·${message.filename}:`,·err)` with `·(err)⏎↹↹↹↹↹↹↹↹console.error(`Error·deleting·file·${message.filename}:`,·err);`
|
|
|
|
Test on Node.js v18.x and OS macOS-latest:
lib/cluster.js#L215
'i' is defined but never used
|
|
Test on Node.js v18.x and OS macOS-latest:
lib/cluster.js#L219
Replace `"hash"` with `'hash'`
|
|
Test on Node.js v18.x and OS macOS-latest:
lib/cluster.js#L219
["hash"] is better written in dot notation
|
|
Test on Node.js v18.x and OS macOS-latest:
lib/cluster.js#L223
Replace `os.tmpdir(),·`metrics-${process.pid}-${message.requestId}.json`` with `⏎↹↹↹↹↹↹↹os.tmpdir(),⏎↹↹↹↹↹↹↹`metrics-${process.pid}-${message.requestId}.json`,⏎↹↹↹↹↹↹`
|
|
Test on Node.js v18.x and OS macOS-latest:
lib/cluster.js#L224
Replace `(err)` with `err`
|
|
|
|
|
|
Test on Node.js v21.x and OS macOS-latest:
lib/cluster.js#L190
Replace `(err)` with `err`
|
|
Test on Node.js v21.x and OS macOS-latest:
lib/cluster.js#L191
Replace `(err)·console.error(`Error·deleting·file·${message.filename}:`,·err)` with `·(err)⏎↹↹↹↹↹↹↹↹console.error(`Error·deleting·file·${message.filename}:`,·err);`
|
|
|
|
Test on Node.js v21.x and OS macOS-latest:
lib/cluster.js#L215
'i' is defined but never used
|
|
Test on Node.js v21.x and OS macOS-latest:
lib/cluster.js#L219
Replace `"hash"` with `'hash'`
|
|
Test on Node.js v21.x and OS macOS-latest:
lib/cluster.js#L219
["hash"] is better written in dot notation
|
|
Test on Node.js v21.x and OS macOS-latest:
lib/cluster.js#L223
Replace `os.tmpdir(),·`metrics-${process.pid}-${message.requestId}.json`` with `⏎↹↹↹↹↹↹↹os.tmpdir(),⏎↹↹↹↹↹↹↹`metrics-${process.pid}-${message.requestId}.json`,⏎↹↹↹↹↹↹`
|
|
Test on Node.js v21.x and OS macOS-latest:
lib/cluster.js#L224
Replace `(err)` with `err`
|
|
|
|
|
|
Test on Node.js v16.x and OS windows-latest:
lib/cluster.js#L190
Replace `(err)` with `err`
|
|
Test on Node.js v16.x and OS windows-latest:
lib/cluster.js#L191
Replace `(err)·console.error(`Error·deleting·file·${message.filename}:`,·err)` with `·(err)⏎↹↹↹↹↹↹↹↹console.error(`Error·deleting·file·${message.filename}:`,·err);`
|
|
|
|
Test on Node.js v16.x and OS windows-latest:
lib/cluster.js#L215
'i' is defined but never used
|
|
Test on Node.js v16.x and OS windows-latest:
lib/cluster.js#L219
Replace `"hash"` with `'hash'`
|
|
Test on Node.js v16.x and OS windows-latest:
lib/cluster.js#L219
["hash"] is better written in dot notation
|
|
Test on Node.js v16.x and OS windows-latest:
lib/cluster.js#L223
Replace `os.tmpdir(),·`metrics-${process.pid}-${message.requestId}.json`` with `⏎↹↹↹↹↹↹↹os.tmpdir(),⏎↹↹↹↹↹↹↹`metrics-${process.pid}-${message.requestId}.json`,⏎↹↹↹↹↹↹`
|
|
Test on Node.js v16.x and OS windows-latest:
lib/cluster.js#L224
Replace `(err)` with `err`
|
|
|
|
|
|
Test on Node.js v21.x and OS windows-latest:
lib/cluster.js#L190
Replace `(err)` with `err`
|
|
Test on Node.js v21.x and OS windows-latest:
lib/cluster.js#L191
Replace `(err)·console.error(`Error·deleting·file·${message.filename}:`,·err)` with `·(err)⏎↹↹↹↹↹↹↹↹console.error(`Error·deleting·file·${message.filename}:`,·err);`
|
|
|
|
Test on Node.js v21.x and OS windows-latest:
lib/cluster.js#L215
'i' is defined but never used
|
|
Test on Node.js v21.x and OS windows-latest:
lib/cluster.js#L219
Replace `"hash"` with `'hash'`
|
|
Test on Node.js v21.x and OS windows-latest:
lib/cluster.js#L219
["hash"] is better written in dot notation
|
|
Test on Node.js v21.x and OS windows-latest:
lib/cluster.js#L223
Replace `os.tmpdir(),·`metrics-${process.pid}-${message.requestId}.json`` with `⏎↹↹↹↹↹↹↹os.tmpdir(),⏎↹↹↹↹↹↹↹`metrics-${process.pid}-${message.requestId}.json`,⏎↹↹↹↹↹↹`
|
|
Test on Node.js v21.x and OS windows-latest:
lib/cluster.js#L224
Replace `(err)` with `err`
|
|
|
|
|
|
Test on Node.js v20.x and OS windows-latest:
lib/cluster.js#L190
Replace `(err)` with `err`
|
|
Test on Node.js v20.x and OS windows-latest:
lib/cluster.js#L191
Replace `(err)·console.error(`Error·deleting·file·${message.filename}:`,·err)` with `·(err)⏎↹↹↹↹↹↹↹↹console.error(`Error·deleting·file·${message.filename}:`,·err);`
|
|
|
|
Test on Node.js v20.x and OS windows-latest:
lib/cluster.js#L215
'i' is defined but never used
|
|
Test on Node.js v20.x and OS windows-latest:
lib/cluster.js#L219
Replace `"hash"` with `'hash'`
|
|
Test on Node.js v20.x and OS windows-latest:
lib/cluster.js#L219
["hash"] is better written in dot notation
|
|
Test on Node.js v20.x and OS windows-latest:
lib/cluster.js#L223
Replace `os.tmpdir(),·`metrics-${process.pid}-${message.requestId}.json`` with `⏎↹↹↹↹↹↹↹os.tmpdir(),⏎↹↹↹↹↹↹↹`metrics-${process.pid}-${message.requestId}.json`,⏎↹↹↹↹↹↹`
|
|
Test on Node.js v20.x and OS windows-latest:
lib/cluster.js#L224
Replace `(err)` with `err`
|
|
|
|
|
|
Test on Node.js v18.x and OS windows-latest:
lib/cluster.js#L190
Replace `(err)` with `err`
|
|
Test on Node.js v18.x and OS windows-latest:
lib/cluster.js#L191
Replace `(err)·console.error(`Error·deleting·file·${message.filename}:`,·err)` with `·(err)⏎↹↹↹↹↹↹↹↹console.error(`Error·deleting·file·${message.filename}:`,·err);`
|
|
|
|
Test on Node.js v18.x and OS windows-latest:
lib/cluster.js#L215
'i' is defined but never used
|
|
Test on Node.js v18.x and OS windows-latest:
lib/cluster.js#L219
Replace `"hash"` with `'hash'`
|
|
Test on Node.js v18.x and OS windows-latest:
lib/cluster.js#L219
["hash"] is better written in dot notation
|
|
Test on Node.js v18.x and OS windows-latest:
lib/cluster.js#L223
Replace `os.tmpdir(),·`metrics-${process.pid}-${message.requestId}.json`` with `⏎↹↹↹↹↹↹↹os.tmpdir(),⏎↹↹↹↹↹↹↹`metrics-${process.pid}-${message.requestId}.json`,⏎↹↹↹↹↹↹`
|
|
Test on Node.js v18.x and OS windows-latest:
lib/cluster.js#L224
Replace `(err)` with `err`
|
|
Test on Bun
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
|
|
Test on Node.js v16.x and OS ubuntu-latest
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v3, actions/setup-node@v3, actions/cache@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
|
|
Test on Node.js v16.x and OS ubuntu-latest:
lib/cluster.js#L190
'err' is already declared in the upper scope on line 183 column 44
|
|
Test on Node.js v20.x and OS ubuntu-latest
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v3, actions/setup-node@v3, actions/cache@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
|
|
Test on Node.js v20.x and OS ubuntu-latest:
lib/cluster.js#L190
'err' is already declared in the upper scope on line 183 column 44
|
|
Test on Node.js v16.x and OS macOS-latest
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v3, actions/setup-node@v3, actions/cache@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
|
|
Test on Node.js v16.x and OS macOS-latest:
lib/cluster.js#L190
'err' is already declared in the upper scope on line 183 column 44
|
|
Test on Node.js v18.x and OS ubuntu-latest
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v3, actions/setup-node@v3, actions/cache@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
|
|
Test on Node.js v18.x and OS ubuntu-latest:
lib/cluster.js#L190
'err' is already declared in the upper scope on line 183 column 44
|
|
Test on Node.js v21.x and OS ubuntu-latest
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v3, actions/setup-node@v3, actions/cache@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
|
|
Test on Node.js v21.x and OS ubuntu-latest:
lib/cluster.js#L190
'err' is already declared in the upper scope on line 183 column 44
|
|
Test on Node.js v20.x and OS macOS-latest
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v3, actions/setup-node@v3, actions/cache@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
|
|
Test on Node.js v20.x and OS macOS-latest:
lib/cluster.js#L190
'err' is already declared in the upper scope on line 183 column 44
|
|
Test on Node.js v18.x and OS macOS-latest
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v3, actions/setup-node@v3, actions/cache@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
|
|
Test on Node.js v18.x and OS macOS-latest:
lib/cluster.js#L190
'err' is already declared in the upper scope on line 183 column 44
|
|
Test on Node.js v21.x and OS macOS-latest
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v3, actions/setup-node@v3, actions/cache@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
|
|
Test on Node.js v21.x and OS macOS-latest:
lib/cluster.js#L190
'err' is already declared in the upper scope on line 183 column 44
|
|
Test on Node.js v16.x and OS windows-latest
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v3, actions/setup-node@v3, actions/cache@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
|
|
Test on Node.js v16.x and OS windows-latest:
lib/cluster.js#L190
'err' is already declared in the upper scope on line 183 column 44
|
|
Test on Node.js v21.x and OS windows-latest
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v3, actions/setup-node@v3, actions/cache@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
|
|
Test on Node.js v21.x and OS windows-latest:
lib/cluster.js#L190
'err' is already declared in the upper scope on line 183 column 44
|
|
Test on Node.js v20.x and OS windows-latest
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v3, actions/setup-node@v3, actions/cache@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
|
|
Test on Node.js v20.x and OS windows-latest:
lib/cluster.js#L190
'err' is already declared in the upper scope on line 183 column 44
|
|
Test on Node.js v18.x and OS windows-latest
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v3, actions/setup-node@v3, actions/cache@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
|
|
Test on Node.js v18.x and OS windows-latest:
lib/cluster.js#L190
'err' is already declared in the upper scope on line 183 column 44
|