Skip to content

githooks: update the release note recommendations#29810

Closed
knz wants to merge 1 commit intocockroachdb:masterfrom
knz:20180907-rn-recommendations
Closed

githooks: update the release note recommendations#29810
knz wants to merge 1 commit intocockroachdb:masterfrom
knz:20180907-rn-recommendations

Conversation

@knz
Copy link
Copy Markdown
Contributor

@knz knz commented Sep 7, 2018

This is the followup to our conversation earlier this week. Independently of the order in which we'll list the sections, I would like to start the motion to:

  • introduce the split into impact and area
  • make recommendations for engineers about the phrasing style.

My draft at a proposal is included here in the form of a PR to our default commit messages. We can iterate on this with the benefit of reviewable and source control.

Should we reach an agreement, this PR alone will not be the end of it: we will also need:

@knz knz requested review from a team and jseldess September 7, 2018 16:10
@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable

@knz knz force-pushed the 20180907-rn-recommendations branch 4 times, most recently from 92267e3 to 7497c23 Compare September 7, 2018 16:52
@knz knz force-pushed the 20180907-rn-recommendations branch from 7497c23 to 2815470 Compare September 7, 2018 17:42
Copy link
Copy Markdown
Contributor

@jseldess jseldess left a comment

Choose a reason for hiding this comment

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

Thanks for moving so quickly on this, @knz. I'm OOO on Monday, but I'll get you thorough comments on Tuesday.

Reviewable status: :shipit: complete! 0 of 0 LGTMs obtained

@tbg tbg added the X-noremind Bots won't notify about PRs with X-noremind label Jun 19, 2019
@knz
Copy link
Copy Markdown
Contributor Author

knz commented Dec 28, 2022

This has been addressed elsewhere since.

@knz knz closed this Dec 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

X-noremind Bots won't notify about PRs with X-noremind

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants