BenDoyle/nomic — pull request #16 by karlhungus (2014-06-16 01:12:20+00:00)
Title: All functionality to rules, add tests, remove unnecessary rules

@BenDoyle @dklassen @prburke
All right think this one is worth merging -- I haven't tested deploy rule (need two up votes). If one of you could put a :+1: on this https://github.com/karlhungus/nomic/pull/14, i'll test deploy rule.

The Good:
- Rename rules to be more obvious
- Move all endpoints to rules (not sure if this is good)
- Make deployment/merge/comment dependant on execution of previous rules, but inside a rule

The Bad:
- Not a lot of tests
- Not sure about the usefulness of having all end points in rules
- Deploy as yet untested
- Ordering totally hard coded and silly
- Removed all but absolutely necessary items (`bad_word_rule`, `haml`, `slim` -- sorry but i judged these as unnecessary to the kernel of nomic, am willing to add these back if people feel strongly like they should be included)