8000 chore: bump node to v22.16.0 (main) by electron-roller[bot] · Pull Request #47217 · electron/electron · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

chore: bump node to v22.16.0 (main) #47217

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

Open
wants to merge 10 commits into
base: main
Choose a base branch
from
Open

Conversation

electron-roller[bot]
Copy link
Contributor

Updating Node.js to v22.16.0.

See all changes in v22.15.1..v22.16.0

Notes: Updated Node.js to v22.16.0.

@electron-roller electron-roller bot requested a review from a team as a code owner May 22, 2025 13:00
@electron-roller electron-roller bot added no-backport semver/patch backwards-compatible bug fixes labels May 22, 2025
@electron-cation electron-cation bot added the new-pr 🌱 PR opened recently label May 22, 2025
@codebytere codebytere self-assigned this May 22, 2025
@electron-cation electron-cation bot removed the new-pr 🌱 PR opened recently label May 23, 2025
@codebytere codebytere requested a review from a team as a code owner May 23, 2025 18:41
@codebytere
Copy link
Member
codebytere commented May 26, 2025

Two failing tests

nodejs/node#57171

parallel/test-sqlite-backup

not ok 2577 parallel/test-sqlite-backup
  ---
  duration_ms: 808.00200
  severity: fail
  exitcode: 1
  stack: |-
    Test failure: 'database backup fails when dest file is not writable'
    Location: test/parallel/test-sqlite-backup.mjs:182:1
    AssertionError [ERR_ASSERTION]: Missing expected rejection.
        at async TestContext.<anonymous> (file:///__w/electron/electron/src/third_party/electron_node/test/parallel/test-sqlite-backup.mjs:188:3)
        at async Test.run (node:internal/test_runner/test:1054:7)
        at async Test.processPendingSubtests (node:internal/test_runner/test:744:7) {
      generatedMessage: false,
      code: 'ERR_ASSERTION',
      actual: undefined,
      expected: [Object],
      operator: 'rejects'
    }
    
    (node:41930) ExperimentalWarning: SQLite is an experimental feature and might change at any time
    (Use `electron --trace-warnings ...` to show where the warning was created)
  ...

and

nodejs/node#56253

parallel/test-config-file

not ok 575 parallel/test-config-file
  ---
  duration_ms: 8333.91000
  severity: fail
  exitcode: 1
  stack: |-
    Test failure: 'should throw an error when the file is non readable'
    Location: test/parallel/test-config-file.js:339:1
    AssertionError [ERR_ASSERTION]: The input did not match the regular expression /Cannot read configuration from node\.config\.json: permission denied/. Input:
    
    ''
    
        at TestContext.<anonymous> (/__w/electron/electron/src/third_party/electron_node/test/parallel/test-config-file.js:348:3)
        at process.processTicksAndRejections (node:internal/process/task_queues:105:5)
        at async Test.run (node:internal/test_runner/test:1054:7)
        at async Test.processPendingSubtests (node:internal/test_runner/test:744:7) {
      generatedMessage: true,
      code: 'ERR_ASSERTION',
      actual: '',
      expected: /Cannot read configuration from node\.config\.json: permission denied/,
      operator: 'match'
    }
    
  ..

Weirdly, I can't get these to fail locally on Linux x64, despite them failing extremely consistently in CI.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no-backport semver/patch backwards-compatible bug fixes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant
0