Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Conversation

clavin
Copy link
Member

@clavin clavin commented Jan 10, 2025

Updating the breaking changes doc in preparation for the release of Electron 34. We identified two changes as being in the wrong sections. In both cases the PR merged after the last release, but was opened before it. Thus, their changes to this doc for the "next" version were no longer current.

I added a "correction" note to the section that we sent out in 33 but actually lands in this upcoming 34 release.

Ref: #43402, #43617

Notes: Corrected the release version of the "menu bar will be hidden during fullscreen on Windows" breaking change.

@clavin clavin added documentation 📓 semver/none target/34-x-y PR should also be added to the "34-x-y" branch. labels Jan 10, 2025
@clavin clavin requested a review from a team as a code owner January 10, 2025 20:04
@electron-cation electron-cation bot added the new-pr 🌱 PR opened recently label Jan 10, 2025
@electron-cation electron-cation bot removed the new-pr 🌱 PR opened recently label Jan 11, 2025
@VerteDinde VerteDinde merged commit e57b69f into main Jan 13, 2025
36 checks passed
@VerteDinde VerteDinde deleted the clavin/update-breaking-changes-e34 branch January 13, 2025 22:15
@release-clerk
Copy link

release-clerk bot commented Jan 13, 2025

Release Notes Persisted

Corrected the release version of the "menu bar will be hidden during fullscreen on Windows" breaking change.

@trop
Copy link
Contributor

trop bot commented Jan 13, 2025

I was unable to backport this PR to "34-x-y" cleanly;
you will need to perform this backport manually.

@trop trop bot added needs-manual-bp/34-x-y and removed target/34-x-y PR should also be added to the "34-x-y" branch. labels Jan 13, 2025
@trop
Copy link
Contributor

trop bot commented Jan 13, 2025

@VerteDinde has manually backported this PR to "34-x-y", please check out #45189

VerteDinde added a commit that referenced this pull request Jan 13, 2025
docs: correct breaking changes versions (#45173)
@trop trop bot added merged/34-x-y PR was merged to the "34-x-y" branch. and removed in-flight/34-x-y labels Jan 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation 📓 merged/34-x-y PR was merged to the "34-x-y" branch. semver/none

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.