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

chore: .gitignore JetBrains, VS Code, Mac files #12

Merged
merged 1 commit into from
Apr 30, 2020

Conversation

mauriciolauffer
Copy link

@mauriciolauffer mauriciolauffer commented Apr 29, 2020

Descrição das Mudanças

Adiciona JetBrains, VS Code, Mac files em .gitignore

Mudanças de comportamento

Ignora arquivos das pastas vscode/ + .DS_Store + .idea/

Testing Procedure

Ao fazer commit/push para o repo, nenhum arquivo das pastas vscode/ + .DS_Store + .idea/ deverao ser enviados.

PR Checklist

  • Possui testes (se não, por favor descreva o motivo de não ter teste)
  • Foi feito Rebase para a master no momento que o PR foi aberto
  • Modificações estão de acordo com o padrão de código
  • Atualizado documentação

PR Checklist do time

  • O PR está apontando para a branch correta
  • Os testes estão passando (ou as falha não são relacionadas a este PR)

closes #13

@rodrigondec rodrigondec requested a review from pictos April 29, 2020 12:27
@rodrigondec
Copy link
Member

@mauriciolauffer

Muito obrigado pela contribuição ❤️

Irei ser 'chato' e reclamar que não existe uma issue aberta para isso. Não estou dizendo que sua contribuição é inválida. Ter isso no gitignore é algo importante e precisamos ter sim.

Mas isso é algo que está sendo discutido no #2 e está explícito no template do PR também. É um padrão que iremos adotar para manter a sanidade dos responsáveis pelo projeto.

Vou pedir para que abra um issue explicando o problema (falta da configuração do gitignore) e depois mencionar esse issue no PR.

Sim, eu sei que é apenas um gitignore e que 'poderíamos' fazer essa exceção. Mas se fizermos exceções teremos que julgar em quais casos ou não vamos fazer exceções. Para evitar isso (e outras situações que podem ser mais complicadas) escolho não fazer exceções.

@mauriciolauffer
Copy link
Author

Criada issue #13

Copy link
Member

@pabrrs pabrrs left a comment

Choose a reason for hiding this comment

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

@mauriciolauffer muito obrigado pela contribuição e por seguir as recomendações do @rodrigondec

💜 🚀 🎉

Copy link
Contributor

@pictos pictos left a comment

Choose a reason for hiding this comment

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

Obg pela contribuição 🚀

@pictos pictos merged commit d0b5e16 into idvogados:dev Apr 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Arquivos locais de configuracao de IDEs podem ser salvos no repo
4 participants