Skip to content

Warning on yarn serve #2522

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
4 of 9 tasks
aocneanu opened this issue Mar 26, 2020 · 9 comments
Closed
4 of 9 tasks

Warning on yarn serve #2522

aocneanu opened this issue Mar 26, 2020 · 9 comments

Comments

@aocneanu
Copy link

Package + Version

  • @sentry/browser
  • @sentry/node
  • raven-js
  • raven-node (raven for node)
  • other:

Version:

 5.15.2

Description

I receive this while serving the app

Module not found: Error: Can't resolve 'perf_hooks' in '/Users/user/code/project/client/node_modules/@sentry/utils/esm'
@joeycozza
Copy link

I believe I'm running into the same underlying issue.

I'm using create-react-app, and @sentry/browser as a dependency. During my compile step, it gives me this error

> react-scripts build
Creating an optimized production build...
Treating warnings as errors because process.env.CI = true.
Most CI servers set it automatically.
Failed to compile.
./node_modules/@sentry/utils/esm/misc.js
Module not found: Can't resolve 'perf_hooks' in '/home/travis/tmp/fresh-cra-template/node_modules/@sentry/utils/esm'```

@mikecousins
Copy link

Yeah this is killing our builds currently. :(

@mikecousins
Copy link

Downgrading to 5.15.1 fixes this.

@carloscorrea93
Copy link

carloscorrea93 commented Mar 26, 2020

I have the same issue in gitlab pipline, downgrade to 5.15.1 solve the issue 😉

@dimaqq
Copy link

dimaqq commented Mar 27, 2020

I got hit by this too.
"perf_hooks" is a node feature, isn't it? So it should not be in prod code... but could be used during build I imagine?
Overall, everything seems to work, it's just that now project doesn't build "cleanly" and thus CI fails 😅

@HazAT
Copy link
Member

HazAT commented Mar 27, 2020

Will be fixed with this PR
#2524

@HazAT
Copy link
Member

HazAT commented Mar 27, 2020

5.15.3 will be released in a few minutes which fixes this issue.

@HazAT HazAT closed this as completed Mar 27, 2020
@Tobbe
Copy link

Tobbe commented Mar 30, 2020

https://docs.sentry.io/error-reporting/quickstart/?platform=browsernpm still needs to be updated, it's saying to install 5.15.2 :)

@kamilogorek
Copy link
Contributor

@Tobbe it's done automatically when we push any changes to the docs themselves. I'll be bumped very soon :)

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

No branches or pull requests

8 participants