Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(recipe): add rubocop-platanus to recipe #402

Merged
merged 2 commits into from
Apr 25, 2022
Merged

Conversation

MrEarle
Copy link
Contributor

@MrEarle MrEarle commented Apr 22, 2022

Contexto

En Platanus es un desafío grande transmitir las decisiones que vamos tomando en relación a estándares y buenas prácticas de desarrollo. Por ej: si empezamos a usar jobs en vez de comandos, ¿cómo hacemos que este cambio se empiece a aplicar en todos los proyectos? o si está prohibido usar una gema x, ¿cómo le avisamos a la gente que use una alternativa?

Basado en esta investigación, descubrimos que es factible transmitir esta información usando reglas custom de Rubocop.

Ya se creó la gema rubocop-platanus como una extension de rubocop con estas reglas custom.

Que se esta haciendo

Se agrega la gema rubocop-platanus y su configuración al recipe de nuevos proyectos.

@MrEarle MrEarle requested a review from difernandez April 22, 2022 17:06
Copy link
Contributor

@difernandez difernandez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice 👏 ! Dejo aprobado, pero agrégate esto en el changelog plis, con un link a este PR

@MrEarle MrEarle merged commit a1549d5 into master Apr 25, 2022
@MrEarle MrEarle deleted the feat/rubocop-platanus branch April 25, 2022 15:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants