diego3g / rsxp-2023

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

fix(mobile): ajuste no job de lint no CI da app mobile e ajustes gerais #91

Closed henriqgoncalvs closed 1 year ago

henriqgoncalvs commented 1 year ago

📋 Descrição

O job que checa a tipagem da app mobile está apresentando um erro. Este erro ocorre pois o tsc está verificando o projeto do expo-router no node_modules, mais especificamente no hook useFocusEffect.

Após investigação e vários testes, descobri que nem a propriedade exclude nem a skipLibCheck servem para definir que o tsc ignore o node_modules, apenas os arquivos de tipo, que nesse caso não se aplica pois é um erro de runtime do typescript, ver na imagem abaixo (se alguém tiver mais alguma sugestão estou aberto).

Este PR é uma sugestão de correção desse erro, porém é importante enfatizar que pode acarretar em um efeito colateral no desenvolvimento da app. Para mais informações sobre a propriedade adicionar, leia a documentação do TS.

Evidências do problema image image

A outra alternativa para resolução desse erro seria remover de fato o job, pois não encontrei uma alternativa fora essa da implementação. Irei abrir uma issue no expo-router, depois linko aqui, pq acho que de fato deve ser algo pra alterar na própria lib.

Aproveitei também pra deixar o nome das runs mais adequadas com o que elas estão fazendo, além de criar o scope de contracts para a nova app que será criada https://github.com/diego3g/rsxp-2023/pull/76

🛠️ Tipo da mudança

🧪 Como isso foi testado?

Print dos jobs rodando no meu fork

image

Print dos jobs rodando nesse PR (verifique abaixo)

image

✅ Checklist: