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

[cmp.categories] Replace 'operator admits' phrasing. #2533

Merged
merged 1 commit into from Aug 18, 2022
Merged

[cmp.categories] Replace 'operator admits' phrasing. #2533

merged 1 commit into from Aug 18, 2022

Conversation

jensmaurer
Copy link
Member

Fixes #2357.

Eelis
Eelis reviewed Nov 27, 2018
View reviewed changes
source/support.tex Outdated Show resolved Hide resolved
@W-E-Brown
Copy link
Contributor

W-E-Brown commented Nov 28, 2018 via email

@jensmaurer jensmaurer added the decision-required A decision of the editorial group (or the Project Editor) is required. label Nov 28, 2018
@jensmaurer
Copy link
Member Author

@W-E-Brown, the concern here is that "an operator" admits something, but it seems we should talk about the type having certain properties here, not the operator.

@jensmaurer
Copy link
Member Author

Editorial meeting: Postpone due to concurrent LWG work in this area.

@jensmaurer jensmaurer removed the decision-required A decision of the editorial group (or the Project Editor) is required. label Jun 5, 2019
@jensmaurer jensmaurer added the needs rebase The pull request needs a git rebase to resolve merge conflicts. label Nov 26, 2019
@jensmaurer jensmaurer added decision-required A decision of the editorial group (or the Project Editor) is required. and removed needs rebase The pull request needs a git rebase to resolve merge conflicts. labels Jun 15, 2021
jensmaurer
jensmaurer commented Jul 16, 2021
View reviewed changes
source/support.tex Outdated Show resolved Hide resolved
@jensmaurer jensmaurer added the changes requested Changes to the wording or approach have been requested and not yet applied. label Jul 16, 2021
@jensmaurer
Copy link
Member Author

jensmaurer commented Jul 16, 2021 •
edited

Also drop "intent" if possible.

"is intended to support" -> admits

@jensmaurer jensmaurer removed the decision-required A decision of the editorial group (or the Project Editor) is required. label Jul 16, 2021
@jensmaurer jensmaurer removed the changes requested Changes to the wording or approach have been requested and not yet applied. label Jul 19, 2021
@wg21bot wg21bot added the needs rebase The pull request needs a git rebase to resolve merge conflicts. label Feb 22, 2022
@jensmaurer jensmaurer removed the needs rebase The pull request needs a git rebase to resolve merge conflicts. label Feb 22, 2022
CaseyCarter
CaseyCarter suggested changes Feb 22, 2022
View reviewed changes
source/support.tex Outdated Show resolved Hide resolved
source/support.tex Outdated Show resolved Hide resolved
@jensmaurer
Copy link
Member Author

@tkoeppe, what about this one?

@tkoeppe
Copy link
Contributor

tkoeppe commented Aug 18, 2022

Looks good, thanks!

@tkoeppe tkoeppe merged commit 887c033 into cplusplus:main Aug 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Eelis Eelis Eelis left review comments

@CaseyCarter CaseyCarter CaseyCarter approved these changes

@tkoeppe tkoeppe tkoeppe left review comments

Assignees

@tkoeppe tkoeppe

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Wonky wording: operators that "admit" other operators?
6 participants
@jensmaurer @W-E-Brown @tkoeppe @Eelis @CaseyCarter @wg21bot