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.

Sign up for GitHub

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

Jump to bottom

Adds CONTRIBUTING file #10

Merged
merged 4 commits into from
Apr 28, 2020
Merged

Adds CONTRIBUTING file #10

merged 4 commits into from
Apr 28, 2020

Conversation

Gerlison
Copy link
Contributor

Descrição das Mudanças

Adicionado o arquivo CONTRIBUTING.md

Bugs Corrigidos

[#1 ]

Mudanças na API

Nenhuma.

Mudanças de comportamento

Nenhuma.

Testing Procedure

Nenhum.

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

Não possui teste devido ser adição de 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)

@Gerlison Gerlison requested review from rodrigondec and pictos April 28, 2020 11:47
rodrigondec
rodrigondec suggested changes Apr 28, 2020
View reviewed changes
Copy link
Contributor

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

Muito obrigado por sua contribuição. Segue algumas alterações, sugestões e questionamentos.

Sinta livre para discordar e discutir 🚀

CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
@Gerlison @rodrigondec
Update CONTRIBUTING.md
cf139fd 
Add code of conduct's link;
Change branch 'develop' to 'dev';
Change some words;

Co-Authored-By: Rodrigo Castro <[email protected]>
@Gerlison Gerlison force-pushed the feature/contributing branch from b07e475 to cf139fd Compare April 28, 2020 12:51
@Gerlison Gerlison requested a review from rodrigondec April 28, 2020 12:52
rodrigondec
rodrigondec suggested changes Apr 28, 2020
View reviewed changes
CONTRIBUTING.md Outdated Show resolved Hide resolved
@Gerlison Gerlison requested a review from rodrigondec April 28, 2020 19:11
rodrigondec
rodrigondec previously approved these changes Apr 28, 2020
View reviewed changes
pictos
pictos approved these changes Apr 28, 2020
View reviewed changes
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.

@Gerlison muito obg pela contribuição, algumas observações. 🚀

CONTRIBUTING.md Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated
- [_How to Write a Git Commit Message_](https://chris.beams.io/posts/git-commit/)
- Seja breve e suscinto no título das mensagens do _commit_.
- Use o corpo da mensagem para explicações adicionais.
- Se esforce para manter a **dev** o mais limpo possível.
Copy link
Contributor

Choose a reason for hiding this comment

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

Isso não é tarefa da pessoa que vai contribuir e sim dos líderes técnicos. Então acredito que isso pode ser removido

Copy link
Contributor

@rodrigondec rodrigondec Apr 28, 2020 •
edited
Loading

Choose a reason for hiding this comment

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

Talvez mover essas coisas para um arquivo interno nosso de 'instruções' para os tech leads.

Poderíamos deixar esses arquivos expostos no repositório por exemplo. Facilita se tiver transição de tech leads (os arquivos que ele precisa ler e prestar atenção já são públicos).

E traria mais transparência do processo de gerência para quem esta contribuindo (ter uma gerência mística nem sempre é bom).

O que acham?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Acho uma boa. Você poderia abrir a issue para tal?

Copy link
Contributor

Choose a reason for hiding this comment

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

Acredito que uma nova issue seria uma boa.

Copy link
Contributor

Choose a reason for hiding this comment

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

Já 'temos' uma issue para isso. A #2 discute sobre o que é um PR válido (algo de responsabilidade dos tec leads).

Vou colocar na descrição da issue essa sugestão. Acho bom esperarmos os tech leads antes de tomar essa decisão. Ou então envolver outras pessoas (@pabrrs, @vilchesalves, @pictos) na decisão.

CONTRIBUTING.md Outdated Show resolved Hide resolved
@pictos pictos self-requested a review April 28, 2020 22:51
pictos
pictos suggested changes Apr 28, 2020
View reviewed changes
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.

apertei errado 🙈

@Gerlison Gerlison force-pushed the feature/contributing branch from d57a846 to a39ee42 Compare April 28, 2020 23:11
@Gerlison @pictos
Update CONTRIBUTING.md
b4de5e9 
Co-Authored-By: Pedro Jesus <[email protected]>
@Gerlison Gerlison force-pushed the feature/contributing branch from a39ee42 to b4de5e9 Compare April 28, 2020 23:14
@Gerlison Gerlison requested a review from pictos April 28, 2020 23:15
@pictos pictos added the Status: NÃO MERGEAR! Mesmo que tudo esteja verde, não faça o merge deste PR. label Apr 28, 2020
@pictos pictos changed the base branch from master to dev April 28, 2020 23:27
@pictos pictos removed the Status: NÃO MERGEAR! Mesmo que tudo esteja verde, não faça o merge deste PR. label Apr 28, 2020
pictos
pictos approved these changes Apr 28, 2020
View reviewed changes
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.

Acredito que está tudo ok. @Gerlison obg pelo PR. E um lembrete amigável, parece que seu cliente git está com o force-push ativado. Isso dificulta a fazer o review, pois não há um diff, se puder desativá-lo nos próximos PRs seria muito bom. Mais um vez obg <3

@pictos pictos requested a review from rodrigondec April 28, 2020 23:30
@pictos pictos merged commit 622a580 into idvogadosorg:dev Apr 28, 2020
@Gerlison
Copy link
Contributor Author

Gerlison commented Apr 28, 2020 •
edited
Loading

Acredito que está tudo ok. @Gerlison obg pelo PR. E um lembrete amigável, parece que seu cliente git está com o force-push ativado. Isso dificulta a fazer o review, pois não há um diff, se puder desativá-lo nos próximos PRs seria muito bom. Mais um vez obg <3

@pictos O force-push foi devido a um squash que fiz em alguns commits automaticos aqui do comentarios. Por terem a mesma mensagem e alterarem apenas um linha ou palavra, juntei em apenas 1 pra facilitar a review.

@rodrigondec rodrigondec mentioned this pull request Apr 28, 2020
Open
@Gerlison Gerlison deleted the feature/contributing branch April 28, 2020 23:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pictos pictos pictos approved these changes

@rodrigondec rodrigondec rodrigondec approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Gerlison @rodrigondec @pictos