Skip to content
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

dependency: electron 33 upgrade #30716

Merged
merged 8 commits into from
Dec 6, 2024

Conversation

mschile
Copy link
Contributor

@mschile mschile commented Dec 4, 2024

Additional details

This pull request includes several updates to dependencies, configurations, and CI workflows to support the upgrade of Electron and Node.js versions. The most important changes include updating the Electron version, Node.js version, Docker images, and related configurations.

Dependency and Version Updates:

  • Updated Node.js version from 20.18.0 to 20.18.1 in various files, including .node-version, package.json, and issue templates. [1] [2] [3] [4] [5]
  • Upgraded Electron version from 32.2.0 to 33.2.1 in package.json. [1] [2]
  • Updated Docker images to use the latest versions with Node.js 20.18.1 and updated browsers in .circleci/workflows.yml and docker-compose.yml. [1] [2] [3] [4] [5]

Configuration Updates:

  • Adjusted the better-sqlite3 prebuild command to target Electron 33.2.1 in .circleci/workflows.yml.
  • Updated the normalizeStdout function to handle new height dimension offsets for Firefox version 133 in system-tests/lib/normalizeStdout.ts. This change is not related to the electron upgrade but since I updated the Firefox version in the docker images, the screenshot dimensions in the system-tests changed. I've logged screenshot height dimensions changed in firefox 133 #30721 to investigate further.

Steps to test

How has the user experience changed?

n/a

PR Tasks

@mschile mschile marked this pull request as ready for review December 4, 2024 20:26
@mschile mschile changed the title feat: electron 33 upgrade dependency: electron 33 upgrade Dec 4, 2024
Copy link

cypress bot commented Dec 4, 2024

cypress    Run #58743

Run Properties:  status check passed Passed #58743  •  git commit 50a4997433: remove firefox 130 changes
Project cypress
Branch Review ryanm/chore/electron-33-upgrade
Run status status check passed Passed #58743
Run duration 17m 22s
Commit git commit 50a4997433: remove firefox 130 changes
Committer Matthew Schile
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 1
Tests that did not run due to a developer annotating a test with .skip  Pending 28
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 765
View all changes introduced in this branch ↗︎
UI Coverage  66.27%
  Untested elements 26  
  Tested elements 55  
Accessibility  96.2%
  Failed rules  0 critical   4 serious   1 moderate   0 minor
  Failed elements 198  

@mschile mschile force-pushed the ryanm/chore/electron-33-upgrade branch from 064a1f5 to e4d80f0 Compare December 5, 2024 16:44
scripts/binary/trigger-publish-binary-pipeline.js Outdated Show resolved Hide resolved
system-tests/lib/normalizeStdout.ts Show resolved Hide resolved
system-tests/lib/normalizeStdout.ts Outdated Show resolved Hide resolved
@mschile mschile requested a review from AtofStryker December 6, 2024 20:33
@mschile mschile merged commit ffefdb0 into release/14.0.0 Dec 6, 2024
123 of 128 checks passed
@mschile mschile deleted the ryanm/chore/electron-33-upgrade branch December 6, 2024 22:24
@jennifer-shehane jennifer-shehane mentioned this pull request Dec 11, 2024
28 tasks
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