servo / highfive

Github hooks to provide an encouraging atmosphere for new contributors
Mozilla Public License 2.0
254 stars 58 forks source link

Check for typos in ini files #57

Open jdm opened 8 years ago

jdm commented 8 years ago

Specifically, it's easy to write disable: whatever instead of the correct disabled: whatever. Also, we could check for an issue number or URL in the disabling message.

Ms2ger commented 8 years ago

Maybe wptrunner should be stricter. We could just whitelist "bug". @jgraham?

jgraham commented 8 years ago

Well there are already more than just disabled, bug and expected, so I'm not sure I want to make a whitelist that has to be updated for any new feature or ad-hoc convention.