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

[container.requirements] Use bulleted lists to introduce identifiers #4499

Merged
merged 1 commit into from Feb 19, 2021

Conversation

jensmaurer
Copy link
Member

@jensmaurer jensmaurer commented Feb 18, 2021

This is a first step towards dissolving the container requirements tables. This step is clearly editorial.

Fixes #1309

@tkoeppe
Copy link
Contributor

tkoeppe commented Feb 18, 2021

@jwakely, @CaseyCarter: could you perhaps take a look at this? Jens and I thought that those introductions of "meaningful names for the purpose of stating requirements" would look much more manageable as a list rather than as an every-expanding sentence.

Copy link
Contributor

@CaseyCarter CaseyCarter left a comment

Choose a reason for hiding this comment

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

Overall: Yes, please! I think LWG has discussed doing this at least once before. I have one nitpick with the bulleting, which is far fewer than the number of problems I have with the pre-existing wording.

source/containers.tex Show resolved Hide resolved
@tkoeppe
Copy link
Contributor

tkoeppe commented Feb 18, 2021

@CaseyCarter: yes, we have long wanted to do something about the tables, and this is only the very first step. Stay tuned for more! :-)

source/containers.tex Outdated Show resolved Hide resolved
@CaseyCarter
Copy link
Contributor

@CaseyCarter: yes, we have long wanted to do something about the tables, and this is only the very first step. Stay tuned for more! :-)

I was referring specifically and only to the bulleted-list-of-names change. We only dislike the paragraphs of names. The tables themselves have a much more storied history of hatred and vitriol. (I've starred the notification for the table PR in my inbox to remind me to take a look at it over the weekend.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Consider using more "Descriptive variable definitions" tables for requirements
3 participants