Skip to content

Conversation

@phillipj
Copy link
Member

Just as with #3343, v4.2.1 should probably also be marked as LTS rather than Stable.

Also what @jasnell decided when creating release blog post on new.nodejs.org/59411.

Needs to land on master and v4.x if it looks good

@cjihrig
Copy link
Contributor

LGTM

@mscdexmscdex added the doc Issues and PRs related to the documentations. label Oct 14, 2015
@jasnell
Copy link
Member

Strange. I could have sworn I'd already done that. Ah well, thanks @phillipj ! LGTM

silverwind pushed a commit that referenced this pull request Oct 14, 2015
PR-URL: #3360 Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Roman Reiss <[email protected]>
@silverwind
Copy link
Contributor

Thanks, landed in dae9fae.

@phillipj
Copy link
MemberAuthor

Are you planning to land it in v4.x also, as done with v4.2.0 / 102cb72@silverwind?

silverwind pushed a commit that referenced this pull request Oct 15, 2015
PR-URL: #3360 Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Roman Reiss <[email protected]>
@silverwind
Copy link
Contributor

Done in b884899.

(Hope I did that right, had to modify the patch slightly so it applied)

@phillipjphillipj deleted the lts-in-changelog branch October 15, 2015 08:57
@silverwind
Copy link
Contributor

Found the cause: CHANGELOG.md on master has an empty line above the title, introduced in e4f2952.

cc: @Fishrock123

@MylesBorins
Copy link
Contributor

I think this should be backported

cc: @jasnell@Trott

@silverwind
Copy link
Contributor

In this case, I already did the v4.x backport on request: b884899.

Is it generally prefered that the commiter does the backporting him/herself or should we leave it to @nodejs/release handle it?

@rvagg
Copy link
Member

for now leave it to the @nodejs/release but that'll likely change over time as it becomes increasingly difficult, we'll have to come up with a better strategy when we end up doing lots of conflict resolution

@Fishrock123
Copy link
Contributor

@silverwind Oops. If it's causing issues let's remove that empty line.

@silverwind
Copy link
Contributor

@Fishrock123 can you put up a PR targeting master to sync the changelogs and remove that line?

@Fishrock123
Copy link
Contributor

@silverwind Sure, do you know off-hand what isn't in sync anymore?

@silverwind
Copy link
Contributor

v4.x is missing the 3.3.1 changelog (might be intended?) as well as 81503e5 and 3ef2e4a.

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docIssues and PRs related to the documentations.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants

@phillipj@cjihrig@jasnell@silverwind@MylesBorins@rvagg@Fishrock123@mscdex