Skip to content
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

Remove outdated rule on single quotes in docs #3495

Merged

Conversation

douglasnaphas
Copy link
Contributor

In PR #1874, a mass replacement of angled single quotes ’ with ASCII
single quotes ' was approved. Since then, the docs have contained a
mixture of angled and ASCII single quotes:

$ git log -S’ --since="Mon Aug 1 16:45:58 2016 -0400" --pretty=oneline | wc -l
17

so it does not make sense to state a rule on them.

In PR reduxjs#1874, a mass replacement of angled single quotes ’ with ASCII
single quotes ' was approved. Since then, the docs have contained a
mixture of angled and ASCII single quotes:

```
$ git log -S’ --since="Mon Aug 1 16:45:58 2016 -0400" --pretty=oneline |
wc -l
17
```

so it does not make sense to state a rule on them.
@netlify
Copy link

netlify bot commented Aug 2, 2019

Deploy preview for redux-docs ready!

Built with commit b882ef2

https://deploy-preview-3495--redux-docs.netlify.com

@timdorr
Copy link
Member

timdorr commented Aug 2, 2019

I'm not even sure why we have this paragraph. We don't enforce this. (AFAIK, Prettier can't do this for us yet). Let's just get rid of it entirely.

@timdorr timdorr merged commit 42474f3 into reduxjs:master Aug 2, 2019
@douglasnaphas douglasnaphas deleted the update-contributing-docs-on-typography branch August 2, 2019 16:40
webMasterMrBin pushed a commit to webMasterMrBin/redux that referenced this pull request Aug 21, 2021
* Remove outdated rule on single quotes in docs

In PR reduxjs#1874, a mass replacement of angled single quotes ’ with ASCII
single quotes ' was approved. Since then, the docs have contained a
mixture of angled and ASCII single quotes:

```
$ git log -S’ --since="Mon Aug 1 16:45:58 2016 -0400" --pretty=oneline |
wc -l
17
```

so it does not make sense to state a rule on them.

* Remove typographic stuff we don't use anyways.


Former-commit-id: 99bbd43
Former-commit-id: 4d3298d
webMasterMrBin pushed a commit to webMasterMrBin/redux that referenced this pull request Aug 21, 2021
* Remove outdated rule on single quotes in docs

In PR reduxjs#1874, a mass replacement of angled single quotes ’ with ASCII
single quotes ' was approved. Since then, the docs have contained a
mixture of angled and ASCII single quotes:

```
$ git log -S’ --since="Mon Aug 1 16:45:58 2016 -0400" --pretty=oneline |
wc -l
17
```

so it does not make sense to state a rule on them.

* Remove typographic stuff we don't use anyways.


Former-commit-id: 99bbd43
Former-commit-id: 4d3298d
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants