Guia de estilos about brasilapi HOT 7 CLOSED

Olá, acabei conhecendo o projeto ao acaso e também estou disposto a contribuir.

Concordo com você, o quanto antes "colocarmos" esse linter melhor. Até por que a gente consegue controlar a qualidade do código como um todo. :)

from brasilapi.

Que tal uma enquete?
Meu voto iria pro padrão Airbnb ✋

from brasilapi.

@paulogdm topo também Airbnb, assim volto a me forçar a usar ; no código 😂

@paulo-santana se o linter for implementado depois, inclusive com bloqueio no CI, os PR abertos precisam ter a master merged ou rebased de volta 👍

from brasilapi.

Como funciona a implementação disso num projeto público que tem PRs abertos? Tem que esperar fechar todos pra rodar o eslint?

from brasilapi.

@filipedeschamps faço suas as minhas palavras kkkk, pra usar ; só com linter gritando comigo msm hehehe

@paulogdm curto bastante o Standardjs, mas to curtindo a ideia de ir de Airbnb, faz tempo que não uso esse 😬🚀

Boa galera, topam puxar então um PR que adiciona o linter no repo? 😬

cc: @RaphaelOliveiraMoura @mathleite

from brasilapi.

@lucianopf Com certeza !!

PR aberta #51 , separei ela em dois commits para ficar fácil fazer rollback ou alterar algo caso necessário. Um commit somente com a adição das configurações do linter e outro commit com o auto fix do linter nos arquivos. :)

from brasilapi.

Putz, @filipedeschamps justamente quando devido a ti eu estava pensando em parar de usar.

from brasilapi.