Aprovação para contribuição

[nicholasess]

Olá senhores. Com o inicio do ano e ausência de muitos, acredito que não haja tanta necessidade que todos concordem com adição de itens nos repositórios através das PRs ou algo similar, tendo em vista que isso é uma comunidade. Acredito também que, mesmo que esteja no meio do ano, muitos não vão participar, por dar importância a outras coisas. Então ter duas pessoas pra mais, a atualização pode ser feita, mas dentro das regras do cerebro. O que acham?

@cerebrobr/colaboradores @cerebrobr/editores @vitorbritto

[pedropolisenso]

@nicholasess confesso que não entendi muito bem a ideia sugerida.

[nicholasess]

e agora?

[felipefialho]

Não entendi, hehe

[vitorbritto]

A intenção é agilizar o processo. Havendo a aprovação de um pull request por duas pessoas (ou mais). Sem a necessidade de aguardar a aprovação de pessoa A, B ou C. Temos muitos colaboradores "cadastrados", mas alguns não participam ativamente.

Eu concordo, se for essa a questão. :neckbeard:

O mais importante é que o commit e pull request sejam feitos de acordo com nossos padrões. Qualquer eventualidade, podemos corrigir depois.

Edit: Resumindo, tornar o fluxo mais ativo. =]

[nicholasess]

É essa questão mesmo

[vitorbritto]

Por mim, está ok! :+1:

[pedropolisenso]

Humm saquei a ideia, muito bom essa sugestão. Eu não aprovo nada, mas por mim também está ok :+1: Rsrs

[hugooliveirad]

:+1: Em ter, 10 de fev de 2015 às 12:50, Vitor Britto notifications@github.com escreveu:

Por mim, está ok! [image: :+1:]

— Reply to this email directly or view it on GitHub https://github.com/cerebrobr/cerebro/issues/16#issuecomment-73711095.

[fdaciuk]

Acho massa! Pra facilitar ainda mais, a pessoa que for conferir se o commit está certo, comenta com um "+1". Se tiver 2 "+1", já pode ser feito o merge.

Se tiver alguma coisa errada, comente direto no código do commit (clicando no "+" que aparece no lado esquerdo da linha), colocando "-1".

Assim que tiver resolvido os problemas do commit, e tendo somados os pontos, 2 "+1", já pode ser feito o merge. O que acham?

[nicholasess]

Muito bom, já estou adotando essa prática. +1 na sugestão do @fdaciuk

[pedropolisenso]

Boa @fdaciuk isso agiliza!

[oliveiraev]

Assim que tiver resolvido os problemas do commit, e tendo somados os pontos, 2 "+1", já pode ser feito o merge. O que acham?

:+1:

[pedropolisenso]

Sim @oliveiraev o @fdaciuk chegou a sugerir isso também!

[fdaciuk]

Ele só fez citação do que eu escrevi @pedropolisenso xD

[pedropolisenso]

Putz pode crer, to olhando aqui e programando nem percebi =)

[vitorbritto]

+1 ou :+1: -1 ou :-1:

[fdaciuk]

+1 ou :+1: -1 ou :-1:

Yep!

[raribeiro]

:+1:

[iamfelipemattos]

+1 @fdaciuk

[felipefialho]

+1 @fdaciuk =)

[thulioph]

Também acho interessante que a turma leia mais a documentação porque quem fez, fez com carinho, atenção, tudo bem explicado.. Eu tento ser ativo aqui mas as vezes passa despercebido, lá na doc. tem informando pra pessoa convocar os editores/ajudantes/jah/alá quando enviar um pr mas ninguém faz isso.

Então já que falei demais +1 :+1: @fdaciuk o/

[danilovaz]

:+1: @fdaciuk nós fazemos esse esquema no repo do io.js e funciona muito bem.

[nicholasess]

Obrigado a todos pela discussão!

[fdaciuk]

Essa informação só precisaria ficar registrada em algum lugar, eu acho. Talvez deixar essa issue marcada como TO DO para colocar essa informação no README.

Assim, qualquer pessoa pode conferir e dar +1. Só deixar claro que esse +1 que está sendo dado, é porque ela conferiu o pull request, e está de acordo com o padrão. Tipo, vai deixar o voto de +1 valendo o **\ mesmo, haha! xD

[nicholasess]

E aonde pode ficar esse TO DO? Não existem regras para os colaboradores e editores seguirem, é só para quem está de fora. De fora pra dentro, não de dentro pra fora.

[nicholasess]

Pensei em adicionar no arquivo central do cerebro

Sobre Colaboradores e Editores

Quando receberem pull request e o mesmo estiver dentro das regras de contribuição do cerebro para o repositório em questão. O pull request será aceito, somente se houverem mais de uma aprovação por parte dos colaboradores e editores. A pessoa que for conferir se o commit está certo, comenta com um "+1". Se tiver 2 "+1", já pode ser feito o merge.

Se tiver alguma coisa errada, comente direto no código do commit (clicando no "+" que aparece no lado esquerdo da linha), colocando "-1".

Assim que tiver resolvido os problemas do commit, e tendo somados os pontos, 2 "+1", já pode ser feito o merge.

[fdaciuk]

Acho uma boa! Só tem que fazer um clean code nesse texto ae :P Escrevi só jogando a ideia, tem muita repetição xD

[thulioph]

Concordo com o @fdaciuk, da uma limpada na repetição e põe nesse local proposto mesmo.