Skip to content

FATAL Could not compile template - sentry.client.js: Unexpected token if #81

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

Closed
Jolg42 opened this issue May 7, 2019 · 3 comments
Closed
Labels

Comments

@Jolg42
Copy link

Jolg42 commented May 7, 2019

Version

master

Reproduction link

https://cmty.app/SorryIDontHaveOne

Steps to reproduce

Node.js 10.7.0
"nuxt": "2.6.3"
"@nuxtjs/sentry": "git://github.com/nuxt-community/sentry-module.git"

Deploying on Zeit Now v1 produce an error see "What is actually happening?"

What is expected ?

A successful build

What is actually happening?

> npm run build
> nuxt:sentry 13:58:09 ✔ Started logging errors to Sentry
> 13:58:10 ℹ Production build
> 13:58:10  FATAL  Could not compile template /home/nowuser/src/node_modules/@nuxtjs/sentry/lib/sentry.client.js: Unexpected token if
>   at Promise.all.templateFiles.map (node_modules/@nuxt/builder/dist/builder.js:5959:17)
>    ╰──────────────────────────────────────────��─────────────────────────────────────────╯

Additional comments?

It could be related to merging #75

This bug report is available on Nuxt community (#c58)
@ghost ghost added the cmty:bug-report label May 7, 2019
@pi0
Copy link
Member

pi0 commented May 7, 2019

@Jolg42 Will fix it in short. BTW consider using specific git commits when using git dependency.

@Jolg42
Copy link
Author

Jolg42 commented May 7, 2019

@pi0 Awesome thank you! And yes indeed, I only did that because I was doing a quick test to check if the new version solved my issue getsentry/sentry-javascript#2052
Next time I'll add a ref so it's easier for you as well!

pi0 pushed a commit that referenced this issue May 7, 2019
@pi0
Copy link
Member

pi0 commented May 7, 2019

@Jolg42 should be fixed on master. I've also made tests working again. Good luck :)

@pi0 pi0 closed this as completed May 7, 2019
@rchl rchl added bug and removed cmty:bug-report labels Feb 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants