Dúvidas gerais sobre a contribuição

[ghost]

Eu fiz um PR recentemente aqui e me surgiram muitas dúvidas sobre fazer isso corretamente, sou novo com tudo isso e quero muito contribuir mas tenho bastante dúvidas ainda (inclusivd fico o tempo inteiro inseguro sobre como agir aqui...
E acredito que mais gente também compartilhe algumas das mesmas dúvidas.

As principais são:

O que são aqueles "lint commits e como faz eles"?

As PRs podem ser mais gerais ou são mais específicas? (Por exemplo as páginas de erro que tô fazendo, coloco as páginas de 500, 404, 410, 403 no mesmo PR ou separado (tá só o PR da 404 por enquanto) )

Além do "roteiro" no npm run commit o que mais precisa ter no commit?
Eu mesmo fiz um PR com a 404 mas não sei se está tudo certo.

Português, inglês ou tanto faz?

Posso abrir issues com perguntas "meio bestas" assim?

O que acham de colocar uma explicação de como contribuir com o projeto pra galera (tipo eu kkkk) que tá iniciando e ainda não entende muito bem do processo do github por exemplo?
Eu mesmo poderia fazer mas preciso entender primeiro dkakdka.

[filipedeschamps]

Ótimas perguntas, vamos para as respostas:

O que são aqueles "lint commits e como faz eles"

lint são certas regras que vão verificar se o que você fez está dentro dos padrões estabelecidos por essas regras. Então o código tem lint para o estilo dele (não vai deixar você programar sem ponto e vírgula, por exemplo), e as mensagens de commit também tem lint.

Para criar commits que irão passar pelo lint, digite o comando abaixo:

npm run commit

As PRs podem ser mais gerais ou são mais específicas? (Por exemplo as páginas de erro que tô fazendo, coloco as páginas de 500, 404, 410, 403 no mesmo PR ou separado (tá só o PR da 404 por enquanto) )

Não temos regra para isso por enquanto. Nesse caso eu faria página apenas para 500 e 404 e colocaria dentro do mesmo PR. Inclusive, testei a página de 404 e ficou linda.

Além do "roteiro" no npm run commit o que mais precisa ter no commit?

Acho que nada a mais e o CI vai avisar se está tudo certo. No caso do seu PR #549 infelizmente a Vercel pulou na frente e travou todo o fluxo da Action (que rodaria o lint dos commits). Mesmo eu destravando a Vercel ali, não rodou os checks. Mas sem problemas, eu consigo forçar de novo.

Português, inglês ou tanto faz?

Tudo que estiver dentro do código deve ser em inglês (comentários, variáveis, funções, etc). Toda documentação externa deve ser em português (README, issues e PRs). Os PRs tem a característica de levar seu título para dentro do commit de merge, e não tem problema. Prefiro uma comunicação fluída aqui fora acima de tudo.

Posso abrir issues com perguntas "meio bestas" assim?

Pode e estas perguntas não são bestas.

O que acham de colocar uma explicação de como contribuir com o projeto pra galera (tipo eu kkkk) que tá iniciando e ainda não entende muito bem do processo do github por exemplo?

Acho ótimo!

[aprendendofelipe]

Parabéns @nottja1mmm, ficou muito boa a página 404! 🤩

[ghost]

Obrigado pelas respostas! Ajudou 100%!

Acho ótimo!

@filipedeschamps

Que bomm!
Acha que dá pra colocar uma secção de "como contribuir" no readme ou em algum outro lugar?

[filipedeschamps]

Total! Acho que dá pra inclusive englobar isso aqui: https://github.com/filipedeschamps/tabnews.com.br#commit-das-altera%C3%A7%C3%B5es

[issdomingoss]

@aprendendofelipe Ola, tudo bem? Corrigi uma issue e gostaria de fazer o push do meu codigo, porem diz que nao tenho permissao, pode me ajudar quanto a isso? Primeira vez fazendo contribuicao! Abracos!!!

[kaique-soares]

Olá @issdomingoss, seja muito bem-vindo ao repositório do TabNews 🤝

Suponho que você tenha clonado este repositório diretamente na sua máquina e caso seja isso mesmo, esse erro de permissão é o esperado. Com isso, você precisa fazer um fork deste repositório para sua conta, ou seja, criar uma cópia do TabNews porém na sua conta, onde você tem permissão de escrita.

Após isso, sugiro criar uma nova branch, implementar suas modificações, commitar e abrir uma Pull Request. Daí tu deve conseguir enviar sua contribuição para cá 👍

Qualquer dúvida, não hesite em perguntar, combinado 👍

[aprendendofelipe]

@aprendendofelipe Ola, tudo bem? Corrigi uma issue e gostaria de fazer o push do meu codigo, porem diz que nao tenho permissao, pode me ajudar quanto a isso? Primeira vez fazendo contribuicao! Abracos!!!

Boa @issdomingoss! 💪

É como o @kaique-soares explicou. E tem mais dicas nessa publicação do @rodrigoKulb:

https://www.tabnews.com.br/rodrigoKulb/boas-praticas-para-criar-um-pull-request-no-projeto-tabnews-com-br

Muito bem vido! 🤗

[aprendendofelipe]

Considerando que o @Rafatcb já referenciou essa issue na mensagem #605 (comment) para levarmos ao CONTRIBUTING.md parte do que foi esclarecido aqui, e que não houve novas dúvidas desde março, vou dar essa issue como concluída. 🤝

Muito obrigado pelas contribuições! 💪