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

update @sentry/node #25

Merged
merged 1 commit into from
Apr 20, 2021
Merged

update @sentry/node #25

merged 1 commit into from
Apr 20, 2021

Conversation

annieke
Copy link
Contributor

@annieke annieke commented Apr 19, 2021

updating to latest @sentry/node since the application was issuing a warning, thank you @aandrewww !

@aandrewww
Copy link
Owner

Hello! Thanks for the PR.
Is it compatible with sentry 5.* ?

@annieke
Copy link
Contributor Author

annieke commented Apr 19, 2021

@aandrewww according to this changelog there are no breaking changes, seems like this is the major change

Starting from the version 6.0.0, all SDKs that support sending sessions data will do so by default.

let me know if there's anything you want to test for specifically!

@aandrewww aandrewww merged commit 3210999 into aandrewww:master Apr 20, 2021
@aandrewww
Copy link
Owner

Merged and published! Thanks!!
[email protected]

@annieke annieke deleted the update-sentry branch April 20, 2021 16:03
gakonst added a commit to ethereum-optimism/optimism that referenced this pull request Apr 28, 2021
so that it's compatible with pino-sentry 0.7.0 aandrewww/pino-sentry#25

This was required because the hardhat upgrade removed any older versions
of the dependency which were still around and could be used.
gakonst added a commit to ethereum-optimism/optimism that referenced this pull request Apr 28, 2021
* fix: update hardhat to use correct solc url

https://github.com/nomiclabs/hardhat/issues/1332\#issuecomment-818212424

* fix(core-utils): revert sentry/node version to 6.2.5

so that it's compatible with pino-sentry 0.7.0 aandrewww/pino-sentry#25

This was required because the hardhat upgrade removed any older versions
of the dependency which were still around and could be used.

* chore: add changeset

* ci: allow tests to be run on demand
InoMurko pushed a commit to omgnetwork/optimism that referenced this pull request May 25, 2021
* fix: update hardhat to use correct solc url

https://github.com/nomiclabs/hardhat/issues/1332\#issuecomment-818212424

* fix(core-utils): revert sentry/node version to 6.2.5

so that it's compatible with pino-sentry 0.7.0 aandrewww/pino-sentry#25

This was required because the hardhat upgrade removed any older versions
of the dependency which were still around and could be used.

* chore: add changeset

* ci: allow tests to be run on demand
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 this pull request may close these issues.

2 participants