Prettier setup

[maxymshg]

No description provided.

[h-michael]

we can use multiple language with Travis like this.

language: rust, node_js

[maxymshg]

@h-michael , Thank you.

I looked over TravicCI docs for something like this, but not found anything except simple apt install to get nodejs.

[maxymshg]

@h-michael , Could you please share the link to the documentation where you find this info? I tried and it's failed for me.

[h-michael]

@maxymshg
Sorry, my info seems to be old.
Now, your way seems to be right.
https://docs.travis-ci.com/user/customizing-the-build/#installing-a-second-programming-language

[h-michael]

@maxymshg
Could you apply prettier to existing queries?

[h-michael]

You should update CONTRIBUTING.md with how to install prettier, how to apply prettier.

[maxymshg]

Ok, thank you. I'll update CONTRIBUTING.md soon.

Could you apply prettier to existing queries?

I already run prettier over .json, .graphql files. What else I missed?

[h-michael]

It seems that CI failed because of returned exit 1 by prettier.

[maxymshg]

Yes, I think it's because of this: prettier/prettier#4144 (comment)

[maxymshg]

I've updated CONTRIBUTING.md. What we should do with this fail from prettier?

[maxymshg]

@h-michael or @tomhoule, Could you please assist with next steps?

[tomhoule]

Sorry for the late review! It's a very busy week. The current setup looks great, thanks for the PR! I think it fails as it should already. If you run prettier once with --write the build should pass, right?

[h-michael]

@maxymshg
I create PR to this branch.
After merging it, you can get exit status 0.

[maxymshg]

Thank you for support, PR merged.

[h-michael]

@tomhoule
Now, pass the CI. 🎉

[h-michael]

@tomhoule
Would you resolve conflicts?

[tomhoule]

@h-michael done, let's merge as soon as it passes CI.