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
wants to merge 2 commits into from
Closed

Conversation

jsoref
Copy link

@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
Contributor

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