diego3g / rsxp-2023

Repositório contendo o código do app do RS/XP 2023
MIT License
443 stars 85 forks source link

[0001] Configuração de workflows de CI para linting e type checking #52

Closed henriqgoncalvs closed 1 year ago

henriqgoncalvs commented 1 year ago

Descrição 📝

Como discutido em https://github.com/diego3g/rsxp-2023/discussions/26, seria importante uma padronização no código que será mergeado na main.

Objetivo 🎯

Comportamento esperado 👁️

Ao fazer um PR e mergear na master, esse job deve rodar passar com 100% de checks.

henriqgoncalvs commented 1 year ago

WIP

LuizHAP commented 1 year ago

@hnqg curti esse padrão de Issue, podíamos colocar aqui no repo né?

henriqgoncalvs commented 1 year ago

@hnqg curti esse padrão de Issue, podíamos colocar aqui no repo né?

Eu acho que pode ficar aberto @LuizHAP, até pq o pessoal já começou a criar as issues. Eu particularmente vou continuar usando esse formato hahah Fica a vontade pra usar tbm

EliasGcf commented 1 year ago

Vi agora que já temos essa issue aqui, mas acabei criando a https://github.com/diego3g/rsxp-2023/issues/66.

Vou deixar abaixo o que comentei por lá:

Faaala, galera!

Vou criar os arquivos de workflows para a execução do CI e proponho duas soluções:

  • Ter somente um arquivo ci.yml para executar as validações nas PRs, independente de qual projeto foi alterado. (mobile, server)
  • Ter dois arquivos: ci-mobile.yml e ci-server.yml, pois assim é possível executar as validações de acordo a qual projeto sofreu alteração. Documentação referente ao filtro de paths: Workflow syntax for GitHub Actions - GitHub Docs#example-including-paths

O que acham?

@hnqg Você já está trabalhando nisso?

henriqgoncalvs commented 1 year ago

Vi agora que já temos essa issue aqui, mas acabei criando a #66.

Vou deixar abaixo o que comentei por lá:

Faaala, galera! Vou criar os arquivos de workflows para a execução do CI e proponho duas soluções:

  • Ter somente um arquivo ci.yml para executar as validações nas PRs, independente de qual projeto foi alterado. (mobile, server)
  • Ter dois arquivos: ci-mobile.yml e ci-server.yml, pois assim é possível executar as validações de acordo a qual projeto sofreu alteração. Documentação referente ao filtro de paths: Workflow syntax for GitHub Actions - GitHub Docs#example-including-paths

O que acham?

@hnqg Você já está trabalhando nisso?

Opa @EliasGcf, tô sim mano. Devo subir a PR agora de noite.

Eu tava pensando em rodar as validações tanto na app mobile quanto server, garantia nunca é demais hahah

EliasGcf commented 1 year ago

Opa @hnqg , entendi.

Imaginando um caso de uma PR que altere somente código no projeto mobile, por exemplo, não vejo necessidade de executar essas mesmas validações no projeto server. Minha opinião apenas.

@diego3g , o que acha?

diego3g commented 1 year ago

@EliasGcf @hnqg acho que podemos executar somente se houver alterações mesmo :)