Skip to content

LWG Motion 18: P1960R0 NB Comment Changes Reviewed by SG1 #3477

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

Merged
merged 1 commit into from
Nov 24, 2019

Conversation

burblebee
Copy link
Contributor

@burblebee burblebee commented Nov 16, 2019

NB US 355, US 358, US 359, US 356, US 364 (C++20 CD):
P1960R0 NB Comment Changes Reviewed by SG1

Fixes #3420.

Issues: Is the commit message correct? I did my best to follow our guidelines.

Fixes cplusplus/nbballot#351
Fixes cplusplus/nbballot#352
Fixes cplusplus/nbballot#354
Fixes cplusplus/nbballot#355
Fixes cplusplus/nbballot#359

@jensmaurer
Copy link
Member

We need the paper number + title first (as-if this were a regular commit), and in the body:
"Also fixes NB ..."
Richard prefers a ", and" before the last NB comment reference.

@jensmaurer jensmaurer added this to the post-2019-11 milestone Nov 16, 2019
@burblebee
Copy link
Contributor Author

@jensmaurer

We need the paper number + title first (as-if this were a regular commit), and in the body:
"Also fixes NB ..."
Richard prefers a ", and" before the last NB comment reference.

Ok, thanks for letting me know. Unless directed otherwise, I'll assume @zygoloid will take care of this when he merges.

@zygoloid zygoloid force-pushed the motions-2019-11-lwg-18 branch from 433c830 to e5363ac Compare November 23, 2019 01:33
Also fixes NB US 355, US 358, US 359, US 356, and US 364 (C++20 CD).
@zygoloid zygoloid force-pushed the motions-2019-11-lwg-18 branch from e5363ac to d7d681f Compare November 24, 2019 03:12
@zygoloid zygoloid merged commit 190577a into master Nov 24, 2019
@jensmaurer jensmaurer deleted the motions-2019-11-lwg-18 branch February 18, 2020 20:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment