Skip to content
This repository was archived by the owner on Sep 27, 2023. It is now read-only.

Fix samples readme reference#423

Closed
jsoref wants to merge 2 commits intogoogleapis:masterfrom
jsoref:samples-readme
Closed

Fix samples readme reference#423
jsoref wants to merge 2 commits intogoogleapis:masterfrom
jsoref:samples-readme

Conversation

@jsoref
Copy link
Copy Markdown

@jsoref jsoref commented Mar 6, 2020

While samples is plural, it's actually the "samples directory" directory which would be singular (and thus if you're going to use an apostrophe, it'd be samples's. But in normal speech, we talk about files in a directory just as such... It's simpler to omit the possessive.

Beyond that, markdown can link to files, which makes things better.

Note: if you actually mean "each sample's readme" then it would be best to use the word each, as otherwise the most likely interpretation is the one I'm making, or at the very least it's definitely ambiguous.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Mar 6, 2020
@bcoe bcoe closed this Mar 9, 2020
@bcoe
Copy link
Copy Markdown

bcoe commented Mar 9, 2020

This file is auto generated from the template here.

I was thinking of this as being the README file that the samples have, vs., the README file at the top level, so I'm also not 100% that it's worth the trouble of dropping the possessive (as it would require updating 80+ README documents).

It might be worth just opening an issue, and when we make a larger edit to the README we coudl pull this in.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants