Js linting

[chrismiceli]

Best to review this by commits. They are distinct.

The first commit upgrades packages and node versions, we should probably accept this commit.

The second commits removes the basic format checks our current linter did and replaces them with prettier, an opinionated (although slightly configurable formatter) that is common in the industry. I've done some basic config to minimize the changes, but it does touch quite a few of our js files. This commit is up for discussion if this is something we want.

[chrismiceli]

I'm fine with this. Can you also write something in SETUP/CODE_STYLE.md about prettier?

Great reminder. I've updated that documentation.

[cpeel]

@chrismiceli - can you please squash the branch so it has two commits:

1. update node, deps (current first commit)
2. everything with prettier

Then I'll get it merged in.

[chrismiceli]

@chrismiceli - can you please squash the branch so it has two commits:

1. update node, deps (current first commit)

2. everything with prettier

Then I'll get it merged in.

Good idea. Rebased off latest master and combined commits.