You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Fazendo a PR #44 e dando uma olhada no código do site percebi que (me corrijam se eu estiver enganado, pois não vasculhei muitas porções de código) não há muitas definições de padrões de código. Para pessoas querendo contribuir, talvez seja interessante deixar claro (talvez no arquivo CONTRIBUTING.md) quais padrões são seguidos.
Padrões como BEM para CSS/SASS, este do Airbnb para javascript, dentre outros tantos já existentes, poderiam ser selecionados e deixados à disposição para que novas contribuições os seguissem.
Entendo que, talvez ao definir tais padrões, o código existente teria que ser refatorado para estar em conformidade, o que seria muito laborioso. Por isso sugiro que, caso seja interessante esta ideia de definir padrões, novas contribuções devam seguí-los, fazendo com que aos poucos boa parte da base entre em concordância com tais definições.
The text was updated successfully, but these errors were encountered:
Fazendo a PR #44 e dando uma olhada no código do site percebi que (me corrijam se eu estiver enganado, pois não vasculhei muitas porções de código) não há muitas definições de padrões de código. Para pessoas querendo contribuir, talvez seja interessante deixar claro (talvez no arquivo
CONTRIBUTING.md
) quais padrões são seguidos.Padrões como BEM para CSS/SASS, este do Airbnb para javascript, dentre outros tantos já existentes, poderiam ser selecionados e deixados à disposição para que novas contribuições os seguissem.
Entendo que, talvez ao definir tais padrões, o código existente teria que ser refatorado para estar em conformidade, o que seria muito laborioso. Por isso sugiro que, caso seja interessante esta ideia de definir padrões, novas contribuções devam seguí-los, fazendo com que aos poucos boa parte da base entre em concordância com tais definições.
The text was updated successfully, but these errors were encountered: