Skip to content

ci: add E44 and E45 to test matrix - #392

Merged
dsanders11 merged 4 commits into
electron:mainfrom
mlaurencin:patch-2
Oct 21, 2026
Merged

dsanders11 merged 4 commits into
electron:mainfrom
mlaurencin:patch-2

Conversation

@mlaurencin

Copy link
Copy Markdown
Member

As in title

@mlaurencin
mlaurencin requested a review from a team as a code owner October 21, 2026 20:08
@dsanders11

Copy link
Copy Markdown
Member

@mlaurencin, needs a lockfile update

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01XniaQor2vcCA65mr3ADVM5
@socket-security

socket-security Bot commented Oct 21, 2026 •

Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Addedelectron@​44.9.310010010098100
Addedelectron@​45.0.010010010098100

View full report

@socket-security

socket-security Bot commented Oct 21, 2026 •

Copy link
Copy Markdown

All alerts resolved. Learn more about Socket for GitHub.

This PR previously contained dependency changes with security issues that have been resolved, removed, or ignored.

Ignoring alerts on:

  • electron@44.9.3
  • electron@45.0.0

View full report

@dsanders11

Copy link
Copy Markdown
Member

@SocketSecurity ignore npm/electron@44.9.3

The test window is created before devtron.install() registers the
renderer preload, so on Electron 44+ the first load can run without
it and the renderer-side IPC tracking tests fail intermittently.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01XniaQor2vcCA65mr3ADVM5
@dsanders11

Copy link
Copy Markdown
Member

@SocketSecurity ignore npm/electron@45.0.0

@dsanders11
dsanders11 enabled auto-merge (squash) October 21, 2026 21:57
@dsanders11
dsanders11 merged commit 52f6890 into electron:main Oct 21, 2026
36 checks passed
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