Use prettier in //third_party/node_modules

[kt3k]

• Added prettier dependency in package.json
• Changed to use local version of prettier in format.py.
• Changed to use .prettierignore for ignoring certain files.

[ry]

Thanks! Looks good - but can you use the --ignore-path CLI option instead of a separate file?

[kt3k]

--ignore-path seems for setting the path of .prettierignore file, not for setting the script path to ignore. I don't find an option to do that in prettier's document...

If we need to avoid .prettierignore, I think we need to do something like:

prettier_targets = glob("js/*.js") + glob("js/*.ts")
prettier_targets.remove("js/msg_generated.ts")
prettier_targets.remove("js/flatbuffers.js")

Do you think it's better than using .prettierignore?

If the separate config file seems a little implicit, how about mentioning it in format.py?

[ry]

LGTM

[ry]

Ok nevermind - thanks for researching it.