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

refactor: adicionado ES6 modules syntax #23

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

diegodario88
Copy link
Contributor

Ao buscar soluções para utilizar a sintaxe de import e outras novidades da linguagem optei por usar o Sucrase, uma alternativa ao Babel que promete ser 20x mais rápido. O acoplamento é simples e a integração com o nodemon funciona muito bem, sem nenhum delay aparente ao transformar CJS para ESM, tendo como possibilidade vários transforms inclusive TypeScript, se caso ele for adotado ao projeto o Sucrase facilitaria a transição.
Ava precisou de um complemento para enxergar os arquivos ESM, para isso foi adicionado a lib esm e a respectiva configuração ao package.json.
Alguns warnings de (param) e ; foram corrigos pelo eslint automaticamente, fora isso procurei não alterar nada além do necessário para concluir a issue closes #22 .

@diegodario88
Copy link
Contributor Author

diegodario88 commented Jun 17, 2020

De acordo com essa (dicussão), signing/verifying/decoding trabalham com operacões totalmente síncronas então no meu entendimento uma opção mais limpa seria remover a callback e tratar eventuais erros com o try/catch. Uma vez que, como esse código faz uso intenso do CPU ele vai cair, invariavelmente, no loop de eventos e bloquear a thread primária então a promessa se torna desnecessária.
Se eu estiver viajando aqui ou se esqueci de algo por favor me de um toque.

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.

1 participant