Add explicit default formatters by language

[imnasnainaec]

A recent VSCode update created a need for formatters to be specified language by language: microsoft/vscode#108447 (comment)

---

This change is 

[johnthagen]

---

package.json, line 16 at r1 (raw file):

    "fmt-frontend": "prettier --write \"./src/**/*.{js,jsx,ts,tsx,json}\"",
    "fmt-frontend-check": "prettier --check \"./src/**/*.{js,jsx,ts,tsx,json}\"",
    "fmt-project": "prettier --write \"./{.vscode,public,scripts}/**/*.{js,jsx,ts,tsx,json}\"",

Can this be merged into fmt-frontend since it's mostly a duplication? I see the distinction, perhaps we should call it fmt-js for (format JavaScript family? including JS, TS, JSON)?

[johnthagen]

Reviewed 4 of 4 files at r1.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @imnasnainaec)

[johnthagen]

---

package.json, line 16 at r1 (raw file):

Previously, johnthagen wrote…

Can this be merged into fmt-frontend since it's mostly a duplication? I see the distinction, perhaps we should call it fmt-js for (format JavaScript family? including JS, TS, JSON)?

Specifically, if we want to validate that everything is formatted properly, we'd need to add a fmt-project-check as well. I'd suggest we just include these extra files in fmt-frontend / fmt-frontend-check. Basically all of these files are part of the frontend project, even though in the Combine we have sorta mashed a lot of projects (including backend) into the top level folder.

[codecov-io]

Codecov Report

Merging #808 (60f584c) into master (0e34593) will increase coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #808      +/-   ##
==========================================
+ Coverage   51.17%   51.18%   +0.01%     
==========================================
  Files         237      237              
  Lines        6449     6449              
  Branches      416      416              
==========================================
+ Hits         3300     3301       +1     
  Misses       2841     2841              
+ Partials      308      307       -1     

Flag	Coverage Δ
backend	55.23% <ø> (ø)
frontend	47.29% <ø> (+0.03%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
src/utilities.tsx	85.00% <0.00%> (+5.00%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0e34593...60f584c. Read the comment docs.

[imnasnainaec]

Reviewable status: 3 of 4 files reviewed, all discussions resolved (waiting on @johnthagen)

---

package.json, line 16 at r1 (raw file):

Previously, johnthagen wrote…

Specifically, if we want to validate that everything is formatted properly, we'd need to add a fmt-project-check as well. I'd suggest we just include these extra files in fmt-frontend / fmt-frontend-check. Basically all of these files are part of the frontend project, even though in the Combine we have sorta mashed a lot of projects (including backend) into the top level folder.

It's just a few files, so I think it's simplest to add them in with fmt-frontend[-check].

[johnthagen]

Reviewed 1 of 1 files at r2.
Reviewable status: complete! all files reviewed, all discussions resolved

[johnthagen]

Reviewed 2 of 2 files at r3.
Reviewable status: complete! all files reviewed, all discussions resolved