Pretty-quick error if JSON file is empty

[valvic66]

Hello,

We created a new package with lerna and storybook that generates an empty config.js by default and pretty-quick crashes in this case.
Is this the way to behave if the file has no code inside?

"prettier": "prettier --write \"packages/**/src/**/*.{ts,tsx,scss,json,js}\"",
"prettyquick": "pretty-quick",

Best regards.

[azz]

Unable to reproduce this. Here's what I tried:

$ yarn init  -y
$ git init
$ echo node_modules > .gitignore
$ yarn add prettier pretty-quick
$ touch x.js
$ git add x.js
$ yarn pretty-quick
🔍  Finding changed files since git revision null.
🎯  Found 1 changed file.
✅  Everything is awesome!
✨  Done in 0.54s.

[valvic66]

Thank you for your reply.
pretty-quick crashes in case of an empty json file (initially mentioned Config.js by mistake).
In our case empty file: Config.json

[azz]

Ah, looks like this an issue with Prettier.
https://prettier.io/playground/#N4Igxg9gdgLgprEAuEACEAaEEAOMCW0AzsqAIYBOFEA7gAqUIkpkBuE+AJpiAEYVkwAazgwAyjkH4oAc2QwKAVzhZpROBRh0BMgLZlkAMzIAbdVgBWRAB4AhAcNFiyuuABlpcI6fMhJFdQpkECtoHhwKaRgAdS4YAAtkAA4ABiwIiHVogRxgiLhA1i8sCjgAR0V8Uu0yPQMkYzMVEHVdfHklZqJpGRM4AEVFCHhvJqwYMl5YzgTkACZxgXwTHoBhCF19YKhoYpBFdQAVSeZG9QBfc6A

I've raised the issue as prettier/prettier#4448.

[valvic66]

Thanks so much. Awaiting a fix.

[azz]

Closing this issue as there's nothing to do in this repo.