Modules

policies that let you slide on common exceptions
prohibit multiline maps, not multistatement maps
empty quotes are okay as the fallback on the rhs of ||
leading zeroes are okay as the first arg to chmod
stringy eval is bad, but it's okay just to "require"
constant value on the right side is ok with 'eq'
ending your package with a simple, fun true value is okay
you can put strict and warnings before "package"