Adding prettier to CI

[gyliu513]

As we have just merged the PR for prettier format at #620 in whole knative/docs repo, but as people will continue all markdown files in development, and will introduce format issue which will violate prettier, how about adding prettier check in CI?

/cc @samodell @RichieEscarez

[samodell]

Hi @gyliu513 , the PR in #620 was generated by a bot that is now run I believe nightly at 6am PT. So any un-pretty changes will eventually be caught and fixed.

Adding @mattmoor to comment on if/when it will be added to the CI process for PRs.

[mattmoor]

Yeah, when Github gives me multi-line suggested edits, this is my plan.

I don't really like the idea of just failing things outright because then everyone needs to learn to run it locally :-/

[gyliu513]

Got it, I think we can continue use the bot. Thanks @samodell @mattmoor

[gyliu513]

@samodell may I know who will maintain the bot? Is it trivial or others? Can you please show more detail for this? I was trying to see if we can leverage same technology for federation v2 document as well at kubernetes-retired/kubefed#478 (comment)

[mattmoor]

There are a bunch of them here: https://github.com/mattmoor/knobots They are based on the knative components themselves. If you are interested I can do a little show and tell.

I hope to find a more permanent home as part of our productivity workgroup (cc @jessiezcc )

[gyliu513]

Ah, thanks @mattmoor , clear now, seems you are leveraging Knative BuildTemplate at https://github.com/mattmoor/knobots/blob/master/config/prettier-template.yaml , this is awesome: Knative is a consumer for Knative ;-)

[jessiezcc]

@chaodaiG will look into this area. knative/test-infra#294 tracks the work needed.