Make quotations in markdown consistent

[cassiewallace]

No description provided.

[levino]

Good contribution. I recommend also making the markdown linting more strict and check for the quotes to be consistent in the future (allow only double quotes). Do you feel like adding this @cassiewallace ?

[levino]

This is helpful but should be enforced in the future by markdown linting.

[cassiewallace]

Ack, will do!

[ericwindmill]

This is actually really important. In past versions of Gatsby (any maybe current), GraphQl queries will fail if there are descrepencies there. Thanks for catching.

[ericwindmill]

@cassiewallace This may be ingornance, but what exactly is "remark-lint-link-title-style"?

[cassiewallace]

@ericwindmill

It's a package that enforces consistency in link and definition titles (NPM link). It is included in some other MD lint presets, such as remark-preset-lint-consistent, but not remark-preset-lint-recommended which is already used in gatsby-starter-docs.

Alternatively, all of the consistent preset could be included here instead, but I chose to use just the part which was called out for this particular issue.

[levino]

The linting plugin does not work for our scenario. It will only check the titles in links, not in the frontmatter entries. See also this issue.

As a side note:

• Even if it worked, it should be a dev dependency, because we do not need it at runtime.
• Frontmatter allows also to not wrap things in quotes at all, maybe that would be even better.

As I do not have write access to this branch: @cassiewallace please remove the commit with the plugin and force push to the branch so the history does not get too cluttered. I will then approve and merge the PR.

[levino]

I gave it a spin by setting one title to use single quotes and then running npm run lint. It did not throw an error, thus the plugin does not work for frontmatter.

[cassiewallace]

Done!

[levino]

Mucho ado... ;) Thanks.