-
Notifications
You must be signed in to change notification settings - Fork 787
layer: add use of "entries" for "files" in tar #282
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
Conversation
|
cc @stevvooe |
|
well, my DCO doesn't suffice for carrying your commit ... |
|
@vbatts rewrite the commit author and force push, it is a trivial addition |
| ### File Types | ||
|
|
||
| Throughout this document section, the use of word "files" includes: | ||
| Throughout this document section, the use of word "files" or "entries" includes: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
“of word” is awkward. Maybe:
Throughout this document section, the words “files” and “entries” include:
I'm also not sure about “document section”. I expect we want something scoped to the whole layer.md, not just the “File Types” h3. How about:
Throughout the layer documentation, the words “files” and “entries” include:
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am going to follow this up
Signed-off-by: Vincent Batts <[email protected]>
|
the "Allow edits from maintainers" is already checked. updated the Author. |
Signed-off-by: Vincent Batts [email protected]