-
Notifications
You must be signed in to change notification settings - Fork 11
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
Sugestão de boilerplate #17
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Obrigado pela contribuição.
Comentei algumas duvídas e sugestões.
Mas gostaria de ressaltar que meu Review não deve ser tomado como indicativo de que todos os códigos JS estão ok 😕
Acredito que estamos quase a ponto de fazer o merge desse PR! 🎉 git reset --soft d0b5e16 isso vai voltar sua branch pro meu último commit, sem excluir as suas modificações, depois disso é só refazer o commit tomando cuidado para colocar as mensagens em português. Ai deixo ao seu critério, se você quer fazer apenas um commit com tudo ou se quer dividir em commits menores. Qualquer dúvida, por favor me avise, ficarei feliz em ajudar! |
@pictos Obrigado pelas instruções, por questão de costume acabo commitando tudo em inglês 😓 Vou me policiar para evitar isso no projeto. As alterações solicitadas foram implementadas 🎉 |
Descrição das Mudanças
Sugestão para estrutura do projeto requerido nas issues #10 e #4
WIP
docs/orientacoes.md
e docs/configuracoes.md`