PR Checklist

• Addresses an existing open issue: n/a
• That issue was marked asaccepting prs
• Steps inCONTRIBUTING.md were taken

There's no issue because this is arequest from Josh.

Overview

The settings.json file contains VSCode settings that are specific to this repository/project.

Typically, in code repos, the IDE ruler is set to the same length as what the code style guide says and/or what the autoformatter is configured to.

For this repository, that value seems to be 80, because of the"printWidth": 80, specification in the ".prettierrc.json" file.

Thus, this project should explicitly represent that in the VSCode settings.

Reason

Having the IDE show a ruler is useful because the autoformatter (in this case Prettier) does not handle formatting comments.
For example, if I am writing a comment like this:

// This is a really long comment that has a lot of verbosity. It's going to explain exactly why I added all of this bullshit code below that at first glance seems unnecessary.functionfoo(){}

Now I've broken the workflow for people who need the code to be at most 80 characters per line. In other words, for people reading the code, comments should be at most 80 characters for line for the same reasons that the code itself should be at most 80 characters per line.

Having the ruler on the screen is a helpful reminder to split up the comment like this:

// This is a really long comment that has a lot of verbosity. It's going to// explain exactly why I added all of this bullshit code below that at first// glance seems unnecessary.functionfoo(){}

(Furthermore, without having the ruler, it's extremely difficult to "properly" split up the long line, since I don't know exactly where the breaks should go.)

Tangent 1

Is there an ESLint rule + autofixer for this? If not, there should be.

Tangent 2

Having the"printWidth": 80, value in the ".prettierrc.json" file is probably a bug, because 80 is already the default value forprintWidth, and thus it doesn't have to be specified at all. It's easier to read a ".prettierrc.json" file if it only specifies changes to the default values. By having it specified at all, it implies that 80 is not the default for Prettier, which is confusing/misleading. I can also make changes to the ".prettierrc.json" file in this PR if you want.