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

chore: add Trop annotations to release notes. #24644

Merged
merged 5 commits into from Jul 21, 2020
Merged

Conversation

ckerr
Copy link
Member

@ckerr ckerr commented Jul 20, 2020

Description of Change

If a change lands in multiple branches, include that information in the release notes. Requested by @electron/wg-qa for the purposes of determining which changes in the release notes are multibranch and which are unique to a particular branch.

Sample output: https://hackmd.io/MONNN0QwRxKoMgAIMROUaw

This also has a happy side-effect: we no longer need the conditionals of showing a release note item based on whether the change is a beta and/or the change was in a previous branch, so this change eliminates a smelly edge case.

CC @electron/wg-qa @electron/wg-releases

Checklist

Release Notes

Notes: none

This is a savepoint commit; still need to display the data.
Trop annotations are in the form of "(Also in 7.3, 8, 9)"
with links to the sibling branches.
@ckerr ckerr requested a review from a team as a code owner July 20, 2020 02:40
@electron-cation electron-cation bot added the new-pr 🌱 PR opened in the last 24 hours label Jul 20, 2020
Copy link
Contributor

@jkleinsc jkleinsc left a comment

Choose a reason for hiding this comment

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

Looks like tests need to be updated to reflect this change.

Copy link
Contributor

@sofianguy sofianguy left a comment

Choose a reason for hiding this comment

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

based on the output, it looks good to me

@electron-cation electron-cation bot removed the new-pr 🌱 PR opened in the last 24 hours label Jul 21, 2020
@ckerr ckerr merged commit b43e601 into master Jul 21, 2020
@release-clerk
Copy link

release-clerk bot commented Jul 21, 2020

No Release Notes

@ckerr ckerr deleted the multibranch-release-notes branch July 21, 2020 15:32
ckerr added a commit that referenced this pull request Jul 21, 2020
Manually backport #24644 to 10-x-y. See that PR for details.
@trop
Copy link
Contributor

trop bot commented Jul 21, 2020

@ckerr has manually backported this PR to "10-x-y", please check out #24666

ckerr added a commit that referenced this pull request Jul 21, 2020
Manually backport #24644 to 10-x-y. See that PR for details.
@trop
Copy link
Contributor

trop bot commented Jul 21, 2020

@ckerr has manually backported this PR to "9-x-y", please check out #24667

@trop trop bot added the in-flight/9-x-y label Jul 21, 2020
ckerr added a commit that referenced this pull request Jul 21, 2020
Manually backport #24644 to 9-x-y. See that PR for details.
@trop
Copy link
Contributor

trop bot commented Jul 21, 2020

@ckerr has manually backported this PR to "master", please check out #24668

@trop
Copy link
Contributor

trop bot commented Jul 27, 2020

@ckerr has manually backported this PR to "8-x-y", please check out #24737

@trop trop bot added the in-flight/8-x-y label Jul 27, 2020
ckerr added a commit that referenced this pull request Jul 27, 2020
Manually backport #24644 to 7-3-x. See that PR for details.
@trop
Copy link
Contributor

trop bot commented Jul 27, 2020

@ckerr has manually backported this PR to "7-3-x", please check out #24738

@trop trop bot added the in-flight/7-3-x label Jul 27, 2020
georgexu99 pushed a commit to georgexu99/electron that referenced this pull request Jul 28, 2020
* chore: add Trop annotations to release notes.

Trop annotations are in the form of "(Also in 7.3, 8, 9)"
with links to the sibling branches.
georgexu99 pushed a commit to georgexu99/electron that referenced this pull request Jul 28, 2020
georgexu99 pushed a commit to georgexu99/electron that referenced this pull request Jul 28, 2020
* chore: add Trop annotations to release notes.

Trop annotations are in the form of "(Also in 7.3, 8, 9)"
with links to the sibling branches.
georgexu99 pushed a commit to georgexu99/electron that referenced this pull request Jul 28, 2020
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.

None yet

4 participants