diorgeneseugenio / fiap-tech-challenge-soat

Esse repositório é usado pelo grupo 30 da primeira turma do curso Pós-Tech FIAP em Arquitetura de software para a realização do Tech Challenge
Apache License 2.0
2 stars 3 forks source link

fix: corrigido build do projeto #29

Closed EltonARodrigues closed 1 year ago

EltonARodrigues commented 1 year ago

Fui fazer um último teste e achei um erro no contêiner, que construía a imagem de produção, mas quando chamava o contêiner ele usava via volume o código do projeto e rodava o npm run dev ao em vez do dist de produção.

Quando corrigi o comportamento descobri que o Build não tava reconhecendo os path customizados do ts e tive que adicionar uma lib pra tratar isso no npm run build

Por último o swagger tava mapeado para pegar as rotas .ts e não estava funcionando no build.

eu mudei o contêiner da aplicação de dev para api no docker compose.

Por favor deem uma última testada nos containers e se o swagger está funcionando normal.

EltonARodrigues commented 1 year ago

O container estava como dev porque é um projeto local, mas como é a entrega do projeto acho que faz sentido mudar mesmo.

Pós-entrega acho que podemos colocar o dev junto no compose

Só achei estranho o lance do tsconfig.build.json. Eu criei esse arquivo só pra excluir do build os arquivos de teste do jest (.spec.ts e .test.ts) porque eles não precisam ir pra produção, teoricamente é a única coisa que esse json tem que fazer... testei tudo aqui excluindo o arquivo novamente e deixando o script de build só com "npx tsc tsconfig.json" e rodou tudo sem erro.

Era isso removi!