Revising changelog

[mhdawson]

Review and make decision on nodejs/node#17062

[MylesBorins]

I'm personally -0. I think that changing the notable changes section makes sense, but the section with commits is for reference, and if we editorialize it makes it harder to actually do comparisons across release lines for commits, where the sha's don't match

[mhdawson]

I'm -0 as well.

[BridgeAR]

Any conclusion here?

[mhdawson]

@nodejs/release any other thoughts?

[gibfahn]

I agree with @MylesBorins in general, although I don't think it makes much difference as the changelogs also include the PR numbers.

[MylesBorins]

I'll move my stance to -1 if it simplifies things Update the notable changes all day, do not change then commit messages for anything other than escaping special characters

…

On Sun, Mar 4, 2018, 12:54 PM Gibson Fahnestock ***@***.***> wrote: I agree with @MylesBorins <https://github.com/mylesborins> in general, although I don't think it makes much difference as the changelogs also include the PR numbers. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#307 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAecV--amPVD9ckT_WEO8FTmLryqsEvQks5tbCpigaJpZM4R9gO1> .

[BridgeAR]

Since this is now a -1 I went ahead and closed the PR. I think this is resolved now and should be closed.

[cjihrig]

I agree with #307 (comment)