BrasilAPI / BrasilAPI/cep-promise

Existencia de "require" - ES Module -> mudar para "import()" - Common JS

Open
#243 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
JavaScript
Stars
3k
Forks
317
PR merge metrics
No merged PRs in 30d

Description

Após atualizar a versão do Nuxt3, o código quebrou no build aparecendo este erro:

```[nuxt] [request error] require() of ES Module ~/nuxt3-app/.output/server/node_modules/node-fetch/src/index.js from ~/nuxt3-app/.output/server/node_modules/cep-promise/dist/cep-promise.min.js not supported.
Instead change the require of index.js in ~/nuxt3-app/.output/server/node_modules/cep-promise/dist/cep-promise.min.js to a dynamic import() which is available in all CommonJS modules.
```

Após verificar o arquivo "cep-promise.js", notei que havia mesmo um "factory(require('node-fetch'))". EU alterei na minha instância para "factory(import('node-fetch'))" e o build completrou sem problemas.

Sugiro que considere verificar se tal issue faz sentido e talvez atualizar o código, pois no build do heroku, ele vai continuar a chamar a versão atual do cep-promise e vai acarretar este erro novamente.
Obrigado
Roberto

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.