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

Sugestão de boilerplate #17

Merged
merged 4 commits into from
May 29, 2020
Merged

Sugestão de boilerplate #17

merged 4 commits into from
May 29, 2020

Conversation

pabrrs
Copy link
Member

@pabrrs pabrrs commented May 3, 2020

Descrição das Mudanças

Sugestão para estrutura do projeto requerido nas issues #10 e #4

WIP

@pabrrs pabrrs marked this pull request as ready for review May 5, 2020 05:16
@pabrrs pabrrs requested review from pictos, rodrigondec, Netaum and tgspn May 5, 2020 05:16
@pabrrs pabrrs added the Status: Precisa Ser Revisado Precisa de code review label May 5, 2020
@pabrrs pabrrs mentioned this pull request May 5, 2020
Copy link
Member

@rodrigondec rodrigondec left a 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 😕

.docker/node-dev.Dockerfile Show resolved Hide resolved
docs/configuracoes.md Outdated Show resolved Hide resolved
.mocharc.yaml Show resolved Hide resolved
.docker/node-prod.Dockerfile Outdated Show resolved Hide resolved
@pictos pictos added the Status: NÃO MERGEAR! Mesmo que tudo esteja verde, não faça o merge deste PR. label May 5, 2020
@pictos pictos removed their request for review May 5, 2020 22:21
@pabrrs
Copy link
Member Author

pabrrs commented May 24, 2020

Foram adicionadas as sugestões da pr #16, feitas pelo @Netaum.

@idvogados/backend

@pabrrs pabrrs requested a review from a team May 24, 2020 15:43
@pictos pictos self-requested a review May 26, 2020 01:04
@pictos
Copy link
Contributor

pictos commented May 26, 2020

Acredito que estamos quase a ponto de fazer o merge desse PR! 🎉
Combinamos que os commits deste repositório serão em português... Sim eu sei que fiz alguns em inglês (shame on me), mas a partir daqui gostaria de ser mais atencioso em relação a isso.
@pabrrs , para reescrever seu commit (pode ser dentro de um só, com todas as modificações daqui) rode esse comandos na sua branch local:

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!

@pabrrs
Copy link
Member Author

pabrrs commented May 26, 2020

@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 🎉

@pabrrs pabrrs merged commit cba80a1 into idvogados:dev May 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: NÃO MERGEAR! Mesmo que tudo esteja verde, não faça o merge deste PR. Status: Precisa Ser Revisado Precisa de code review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants