Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: cra-ts building process #2

Merged
merged 4 commits into from
Aug 8, 2023
Merged

Conversation

guligon90
Copy link
Contributor

Issues associadas

Motivação

Esse hotfix retifica problemas existentes na constituição das dependências subsidiárias do cra-ts. Haja vista que, em tempo de transpilação do projeto, os seguintes erros estavam ocorrendo:

  • Erro em função da falta da dependência react-tables:
    build-error-missing-deps

  • Erro em função de um problema de exportação de módulo utilitário na biblioteca axios-extensions:
    build-error-missing-axios-extensions-export

O que foi feito?

  • Instalação manual das bibliotecas react-tables e @types/react-tables, ambas consumidas pelo omega-react-ui e que não estão no momento incluídas nas dependências de produção da mesma.
  • Em webpack.monkey.js, foi realizada uma inclusão de exportação axios/lib no node_modules, para que o módulo utilitário buildURL possa ser encontrado por outras libs consumidoras. Esse problema é inerente da axios e foi discutido nesse bugfix.
  • Agrupamento do componente de boilerplate App.tsx em uma camada dedicada de componentes.
  • Em package.json, inclusão de novos scripts:
    • Validação de base de código (yarn validate);
    • Transpilação de código TypeScript (yarn build:validate e yarn build);
    • Execução do projeto em modo de produção (yarn prod)

Como testar?

  • Executar os novos scripts de automação de projeto:
    • yarn build:validate deverá transpilar TS, mas não emitir módulos JS;
    • yarn build deverá transpilar TS e gerar saída na pasta build;
    • yarn validate deverá rodar, em sequência, yarn build:validate, yarn lint:format e yarn lint:fix;
    • yarn start:prod deverá rodar o projeto da pasta build;
  • Incluir algum componente mais complexo de omega-react-ui em App.tsx e verificar, em tempo de execução, a estabilidade do projeto.

@guligon90 guligon90 requested a review from kivervinicius August 8, 2023 13:15
@guligon90 guligon90 force-pushed the hotfix/ts-build-process branch from c07260c to 6e7edb6 Compare August 8, 2023 15:37
@omegaeducacional omegaeducacional merged commit cab318e into main Aug 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants