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

use a bulleted list for rules in many places #612

Merged
merged 1 commit into from
Mar 15, 2017

Conversation

erikh
Copy link
Contributor

@erikh erikh commented Mar 14, 2017

Hope this is a welcome change. Thanks!

@jonboulle
Copy link
Contributor

would we want to do this more broadly?

@vbatts
Copy link
Member

vbatts commented Mar 14, 2017

LGTM
and sure it could be applied to other places as well

Approved with PullApprove

@erikh
Copy link
Contributor Author

erikh commented Mar 14, 2017 via email

Signed-off-by: Erik Hollensbe <github@hollensbe.org>
@erikh erikh changed the title annotations.md: use a bulleted list for rules use a bulleted list for rules in many places Mar 14, 2017
@erikh
Copy link
Contributor Author

erikh commented Mar 14, 2017

I've expanded things a bit to include lists in other places. Hopefully it works.

I will do more just at a later point. Please consider this ready for merge unless it needs edits.

@philips
Copy link
Contributor

philips commented Mar 15, 2017

LGTM

Approved with PullApprove

1 similar comment
@vbatts
Copy link
Member

vbatts commented Mar 15, 2017

LGTM

Approved with PullApprove

@vbatts vbatts merged commit f59a917 into opencontainers:master Mar 15, 2017
@jonboulle
Copy link
Contributor

ugh, I'm not so happy about this; it doesn't make sense in sections that are English prose.

@erikh
Copy link
Contributor Author

erikh commented Mar 16, 2017 via email

@erikh erikh deleted the annotations-cleanup branch March 16, 2017 23:09
@vbatts vbatts mentioned this pull request May 19, 2017
@stevvooe
Copy link
Contributor

stevvooe commented Jul 5, 2017

@erikh I actually agree with @jonboulle here. This does not help readability.

@stevvooe
Copy link
Contributor

stevvooe commented Jul 5, 2017

@vbatts Can we revert this?

@vbatts
Copy link
Member

vbatts commented Jul 6, 2017

I guess, but it would be post-v1.0.0

@stevvooe
Copy link
Contributor

stevvooe commented Jul 6, 2017

Seems like this can go in the v1.0.0. This is pretty awful.

@erikh
Copy link
Contributor Author

erikh commented Jul 6, 2017 via email

@stevvooe
Copy link
Contributor

stevvooe commented Jul 6, 2017

@erikh Thanks! But it looks like this will go out with the 1.0.

@erikh
Copy link
Contributor Author

erikh commented Jul 6, 2017 via email

@stevvooe
Copy link
Contributor

stevvooe commented Jul 6, 2017

@erikh Yes, please, but it may not be merged immediately.

@erikh
Copy link
Contributor Author

erikh commented Jul 9, 2017

I'm sorry, I won't have the time to do this, nor will I have much to spend on opencontainers in general.

Sorry if I caused any trouble. Thank you for letting me be involved in the past!

@vbatts
Copy link
Member

vbatts commented Jul 10, 2017

@stevvooe this revert is going to have to be manually done.

@stevvooe
Copy link
Contributor

@vbatts Yes, agreed. I'll take care of it after 1.0.

@vbatts
Copy link
Member

vbatts commented Jul 10, 2017 via email

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.

6 participants