platanus / potassium

A Rails application generator by Platanus, inspired by Suspenders
MIT License
232 stars 17 forks source link

feat(assets-rubocop-yml): enable new rules #429

Closed difernandez closed 1 year ago

difernandez commented 1 year ago

Se habilitan algunas reglas nuevas en rubocop:

Esta última me gustaría que se revise particularmente. Se define un orden en el que van los distintos elementos de una clase, pensado principalmente para el orden en modelos. El orden es una combinación del ejemplo que sale en los docs y lo que vi en platanus revisando modelos. Ahí díganme si les hace sentido o cambiarían algo. El cop define una lista en ExpectedOrder que puede tener: grupos definidos en Categories, macros sueltas (enum, aasm), algunas cosas definidas de antemano en algún lado que rubocop entiende (constants, methods), y cosas anteriores con prefijo de nivel de privacidad.

Al recibir una clase con el orden dado vuelta lanza los siguientes errores:

image

Y al correr el autocorrect queda así:

image

Los espaciados quizás es mejorable, pero hace harto de la pega

difernandez commented 1 year ago

@gmq agregué una regla que me había faltado desde que revisaste por si acaso, la de redundant validation