Skip to content
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

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

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.

Also fixes NB US 355, US 358, US 359, US 356, and US 364 (C++20 CD).
@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