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.

Sign up for GitHub

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

Jump to bottom

kebab-case all the exposition only ranges types #3097

Closed
cjdb opened this issue Jul 29, 2019 · 4 comments · Fixed by #3559
Closed

kebab-case all the exposition only ranges types #3097

cjdb opened this issue Jul 29, 2019 · 4 comments · Fixed by #3559
Assignees
@jensmaurer
Labels
big An issue causing a large set of changes, scattered across most of the text.

Comments

@cjdb
Copy link
Contributor

cjdb commented Jul 29, 2019

LWG asked for the exposition-only types in namespace ranges to universally be changed to kebab-case in Cologne, so that it's easier to tell which types are public-facing, and which are exposition-only.

I've volunteered to make that change, but it's a good idea to make an issue to track it.
@jensmaurer
Copy link
Member

@cjdb, The merge of the motions is now complete, except for the concepts renaming. Now is a good time to make the change and send a pull request.

@jensmaurer
Copy link
Member

@cjdb, any updates here?

@cjdb
Copy link
Contributor Author

cjdb commented Oct 3, 2019

Sorry, I haven't made time for this (but really should). Do you want a single commit, one commit per name change (but as a single PR), or many PRs?

@jensmaurer
Copy link
Member

A single commit sounds fine for me.

@jensmaurer jensmaurer added the big An issue causing a large set of changes, scattered across most of the text. label Dec 15, 2019
@jensmaurer jensmaurer self-assigned this Dec 15, 2019
@jensmaurer jensmaurer mentioned this issue Dec 15, 2019
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jensmaurer jensmaurer

Labels
big An issue causing a large set of changes, scattered across most of the text.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[ranges] Mark exposition-only names as kebab-case. jensmaurer/draft
2 participants
@cjdb @jensmaurer