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 error handling #17

Closed
EvodiaAut opened this issue Jan 15, 2019 · 2 comments · Fixed by #18
Closed

Fix error handling #17

EvodiaAut opened this issue Jan 15, 2019 · 2 comments · Fixed by #18
Milestone

Comments

@EvodiaAut
Copy link

Describe the bug

Error message on run

node_modules/vue-cli-plugin-prerender-spa/index.js:101
  } catch {
          ^

SyntaxError: Unexpected token {

Can you add in Line index.js:101 a exception parameter? Informations

from

} catch { <-------
  if (existsSync(oldConfigPath)) {
    options = JSON.parse(readFileSync(oldConfigPath).toString("utf-8"));
  }
}

to

} catch(ex) { <-------
  if (existsSync(oldConfigPath)) {
    options = JSON.parse(readFileSync(oldConfigPath).toString("utf-8"));
  }
}

Additional context
Package version: 1.1.1
Vue version: 2.5.21
Vue CLI version: 3.3.0

@EvodiaAut
Copy link
Author

I get this error on node version v8.15.0
In node version v10.x is works perfectly

@nahanil nahanil mentioned this issue Jan 15, 2019
2 tasks
SolarLiner pushed a commit that referenced this issue Jan 15, 2019
Running `npm run build` fails with the below error, this ought to stop that from happening (though the `err` variable will go unused).

Fixes #17
@SolarLiner
Copy link
Owner

Hotfix uploaded to npm as v1.1.2, thanks for the notice!

@SolarLiner SolarLiner added this to the 1.1 hotfixes milestone Jan 17, 2019
SolarLiner added a commit that referenced this issue Jan 17, 2019
Release version 1.1.3

- Switch to a Babel setup to support Node.js 8.x #17
- Support Vue CLI 3.3 "publicPath" setting #19
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 a pull request may close this issue.

2 participants