As-Raparigas-do-Codigo / ardc-website

Website d'As Raparigas do Código
https://raparigasdocodigo.pt
MIT License
5 stars 8 forks source link

Add husky to check linting and formatting errors upon committing staged files #128

Closed juliamendesc closed 11 months ago

juliamendesc commented 11 months ago

Descrição

Fixes #115

Como testar esta modificação?

Checklist:

vercel[bot] commented 11 months ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
ardc-website ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 26, 2023 6:50pm
toothlesspanda commented 11 months ago

If it works, parece-me ótimo. Só diria para acrescentares esses comandos ao readme 🙏 se puderes! Obrigada! :D

juliamendesc commented 11 months ago

If it works, parece-me ótimo. Só diria para acrescentares esses comandos ao readme 🙏 se puderes! Obrigada! :D

Tinha feito, mas acho que nos meus testes acabei por apagar. Já está lá (aproveitei também inseri o comando do linter e do format que não estavam) 🙂

juliamendesc commented 11 months ago

Também retirei o package-lock, portanto façam npm i quando derem merge :)

juliamendesc commented 11 months ago

Olhem, fiz merge do dev branch neste e mesmo assim há falha na Vercel. Conseguem me dizer qual é o erro pra que eu possa corrigir? Não tenho acesso 🙈

gustavorodrigues19 commented 11 months ago

@juliamendesc pode atualizar com a dev novamente e fazer o commit do package-lock.json por favor ? (Acho que assim vai passar).

Vou dar o approve pra fazer o merge :)

Novamente, queria agradecer pela sua contribuição e execelente trabalho 🙂 Essa tarefa é meio "chatinha". You rock !!! ❤️

juliamendesc commented 11 months ago

Rapaz, eu e o git nos desentendemos aqui, mas ja resolvi. Já vou fazer o que vc pediu :D

juliamendesc commented 11 months ago

@juliamendesc pode atualizar com a dev novamente e fazer o commit do package-lock.json por favor ? (Acho que assim vai passar).

Vou dar o approve pra fazer o merge :)

Novamente, queria agradecer pela sua contribuição e execelente trabalho 🙂 Essa tarefa é meio "chatinha". You rock !!! ❤️

Fiz isso, mas continuei com erro na vercel. Não consigo ver qual é o erro precisamente. Consegue partilhar um print comigo? 🫠

gustavorodrigues19 commented 11 months ago

@isabelcosta ou @inesmartins conseguem ver o erro na vercel, pls ? Não tenho acessos também :(

toothlesspanda commented 11 months ago

@juliamendesc podes experimentar remover a homepage do package.json sff?

image se abrirem o link do vercel e forem à console aparece isto, tem haver com o cors e as permissoes do vercel provavelmente. eu acrescentei essa homepage porque estava a dar erro em produção, mas isso também não foi o que resolveu e acho que acabou por criar problemas com o vercel

também pode ser o .env.production

juliamendesc commented 11 months ago

@toothlesspanda eu nem chego nessa página pq enfrento um bonito 404 XD

image

mas retirei a homepage do package.json e sigo com o deployment failing. como já tenho o seu print, vou dar uma pesquisada aqui pra tentar perceber esse erro :)

juliamendesc commented 11 months ago

Descobri! XD

era uma variável do eslint que não deixava dar build! coloquei pra mostrar os exemplos do linter e não retirei 🙈

agora tudo certo. Desculpem a bagunça! @toothlesspanda @gustavocr19-developer