Gatsby: [perilbot] validate yaml for different yaml files + required fields

Created on 21 Jul 2018  Â·  6Comments  Â·  Source: gatsbyjs/gatsby

E.g. the site showcase yaml file.

Travis does this but it'd be nicer to put up a comment with the YAML error printed out so it's easier for people to fix. Many people never look at Travis.

maintenance

Most helpful comment

The bot could also post a screenshot to the comment section which would further speed up reviewing sites.

All 6 comments

@calcsam just had a great idea too that Peril could auto-merge site show case PRs. It'd validate that it's a Gatsby site, make sure it isn't a porn site, and then merge it in.

@KyleAMathews @calcsam I can definitely look into this. I have no idea if there's a way to reliably validate that it's not porn, though.

Yeah, it doesn't have to be perfect — or do any checking really at first — we can just manually check them after the fact and revert any that we don't want in the showcase.

I think just checking they're linking to a Gatsby site would be sufficient to start with — which is trivial as grab HTML from site and look for ___gatsby

The bot could also post a screenshot to the comment section which would further speed up reviewing sites.

Ref. #6538

Was this page helpful?
0 / 5 - 0 ratings

Related issues

benstr picture benstr  Â·  3Comments

andykais picture andykais  Â·  3Comments

signalwerk picture signalwerk  Â·  3Comments

3CordGuy picture 3CordGuy  Â·  3Comments

ferMartz picture ferMartz  Â·  3Comments