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

Maximum call stack size exceeded when using in combination with global-prefix #47

Closed
maoberlehner opened this issue Apr 22, 2019 · 1 comment

Comments

@maoberlehner
Copy link

maoberlehner commented Apr 22, 2019

I hope this does not get deleted automatically, but https://nuxtjs.cmty.io/issues/new does not work!

Reproduction:

CodeSandbox: https://codesandbox.io/s/y0yxr7923j (unfortunately I can't get the CodeSandbox to run again after changing the nuxt.config.js and restarting. Please download the CodeSandbox and run locally to reproduce. Thx!)

  1. Install Nuxt
  2. Install and configure node-sass-magic-importer (see nuxt.config.js in CodeSandbox)
  3. Run npm run generate
  4. 100% CPU and nothing else happens (because the developers of global-prefix use try / catch without logging the error).

I've hunted down the error to the global-prefix package to the following lines: https://github.com/jonschlinkert/global-prefix/blob/master/index.js#L65

Add a console.error(err) in the catch section to see the Maximum call stack size exceeded error in node_modules/consola/dist/consola.js. Remove the try / catch block and just return null everything works fine.

Everything worked fine before Nuxt 2.5 so I assume something has changed with this package with the 2.5 release?

Thanks for the great work on Nuxt.js - I'm a big fan!

This question is available on Nuxt community (#c47)
@ghost ghost closed this as completed Apr 22, 2019
@ghost ghost added the cmty:question label Apr 22, 2019
@ghost
Copy link

ghost commented Apr 22, 2019

This issue as been imported as question since it does not respect consola issue template. Only bug reports and feature requests stays open to reduce maintainers workload.
If your issue is not a question, please mention the repo admin or moderator to change its type and it will be re-opened automatically.
Your question is available at https://cmty.app/nuxt/consola/issues/c47.

This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant