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

Update Electron to v26 #30679

Merged
merged 1 commit into from
Aug 21, 2023
Merged

Update Electron to v26 #30679

merged 1 commit into from
Aug 21, 2023

Conversation

gzdunek
Copy link
Contributor

@gzdunek gzdunek commented Aug 18, 2023

I tested it quickly on macOS and didn't find any issues. In the release notes I also don't see any changes that affect us.

This should be backported after we finish the test plan.

@ravicious
Copy link
Member

Do you have a tag build?

@gzdunek
Copy link
Contributor Author

gzdunek commented Aug 18, 2023

No, I don't. I only did a local build, which I ran.
Do you think it is needed? I thought that we could wait for a test plan build.

@ravicious
Copy link
Member

Ah, I keep forgetting that this is something we can do. So yeah, thorough testing is not necessary, but I'd still run a tag build just to make sure that this change won't block the alpha releases.

@gzdunek
Copy link
Contributor Author

gzdunek commented Aug 18, 2023

Copy link
Member

@ravicious ravicious left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tested the local shell and focusing the window for db & kube access. As you said, a more thorough testing can be performed during the test plan.

@gzdunek
Copy link
Contributor Author

gzdunek commented Aug 21, 2023

The build passed.

@gzdunek gzdunek added this pull request to the merge queue Aug 21, 2023
Merged via the queue into master with commit 8e68beb Aug 21, 2023
@gzdunek gzdunek deleted the gzdunek/update-electron-26 branch August 21, 2023 08:05
@public-teleport-github-review-bot

@gzdunek See the table below for backport results.

Branch Result
branch/v11 Create PR
branch/v12 Create PR
branch/v13 Create PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants