Differential D20762

Add an initial RELNOTES file.
ClosedPublic
Actions

Authored by markj on Jun 25 2019, 6:27 PM.

Details

Reviewers

• ian

Commits

rS350089: Add an initial RELNOTES file.

Diff Detail

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 25085
Build 23790: arc lint + arc unit

Event Timeline

markj created this revision.Jun 25 2019, 6:27 PM

Harbormaster completed remote builds in B25043: Diff 59026.Jun 25 2019, 6:27 PM

I love the clean simplicity of this after all the back and forth on mailing lists about markup languages and other complications that were far beyond this original simple vision.

A nit: why 72 columns rather than 80?

This revision is now accepted and ready to land.Jun 27 2019, 1:30 AM

rgrimes added a subscriber: rgrimes.Jun 27 2019, 2:53 AM

In D20762#449524, @ian wrote:

A nit: why 72 columns rather than 80?

Hmm, for some reason I'm used to keeping text to 72 columns but I don't have a concrete justification for doing that. I'll change it to 80.

Reformat to 80 columns.

This revision now requires review to proceed.Jun 27 2019, 3:55 PM

Harbormaster completed remote builds in B25085: Diff 59104.Jun 27 2019, 3:55 PM

Remove unintended change to UPDATING.

Harbormaster completed remote builds in B25086: Diff 59105.Jun 27 2019, 3:56 PM

In D20762#449673, @markj wrote:

Reformat to 80 columns.

One reason for keeping it at 72 columns is so that email replies can often stay under the 80 column width, not really important here. Also since you have eaten 8 columns off the front for the text it makes it very balanced to eat 8 off the end, but again, not realy important here. This text is probably going to get re-flowed when it ends up in the release notes anyway.

In D20762#449693, @rgrimes wrote:

In D20762#449673, @markj wrote:

Reformat to 80 columns.

One reason for keeping it at 72 columns is so that email replies can often stay under the 80 column width, not really important here. Also since you have eaten 8 columns off the front for the text it makes it very balanced to eat 8 off the end, but again, not realy important here. This text is probably going to get re-flowed when it ends up in the release notes anyway.