codeforamerica / brigade-information

Data about Code for America brigades and other civic tech organizations for the CfA API
MIT License
17 stars 63 forks source link

'format-json' still needed? #112

Closed Shaunm44 closed 4 years ago

Shaunm44 commented 4 years ago

Is this line from the readme still accurate?

Before committing your change, please make sure that there are no formatting issues by running the bin/format-json script. (You will need to brew install jq moreutils for that script to run.)

I ran the exec, but the format of whitespace was shifted for the whole doc.

tdooner commented 4 years ago

Hm, interesting. What changes specifically are you seeing? And, what version of jq do you have? (jq --version)

Shaunm44 commented 4 years ago

jq-1.6

Shaunm44 commented 4 years ago

I'm going to close this issue. The formatting issue was due to VSCode configs. 'format-json' works as described. My issue was between keyboard and chair 🙃