max-mapper / max-mapper/javascript-for-cats

change to single quote strings

Open
#45 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
JavaScript
Stars
794
Forks
139
PR merge metrics
No merged PRs in 30d

Description

nice work dude!

It would be great if the use of single or double quotes strings could be normalized.

IMO, all to single quotes, so we can set a good start point for beginners. Indeed you recommend https://github.com/feross/standard which states the single quote style.

Rules

2 spaces – for indentation
Single quotes for strings – except to avoid escaping

for example from

var firstCat = { name: "bill", lastName: "the cat", address: "The Alley" }

to

var firstCat = { name: 'bill', lastName: 'the cat', address: 'The Alley' }

If you like the idea, I'll be glad to help with the change and make a PR.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Inspect the repository's JavaScript examples and compare their string quoting with the requested single-quote style. Normalize applicable examples while retaining double quotes where escaping would be avoided, then verify that the documented snippets consistently follow the stated rule.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript
Domain
documentation
Issue type
Documentation
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.