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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Announcement: We're on the v7 Road! #4882

Closed
Lms24 opened this issue Apr 7, 2022 · 6 comments 路 Fixed by #4876
Closed

Announcement: We're on the v7 Road! #4882

Lms24 opened this issue Apr 7, 2022 · 6 comments 路 Fixed by #4876

Comments

@Lms24
Copy link
Member

Lms24 commented Apr 7, 2022

Hi everyone!

We wanted to notify you all that we're currently working actively towards the release of our next major version of the JavaScript SDKs - v7 馃殌

This means that we're temporarily pausing work on master and we're focussing on incorporating all (breaking) changes that will be introduced with v7. Feel free to take a look at our v7 roadmap (#4240).

Also, check out our v7 progress here: master...7.x

Migration guide: https://github.com/getsentry/sentry-javascript/blob/7.x/MIGRATION.md

Over the course of the next weeks we'll be releasing a few v7 beta versions for everyone to try out. As always, please feel free to submit feedback. In the meantime, we'll do our best to answer your issues and questions as usual but we ask for your understanding that it might take longer than usual before we get back to you. We'll work on non-critical issues and changes once v7 is released.

Thank you for your support and we're looking forward to receiving your feedback and input from the v7 releases!

@Lms24 Lms24 pinned this issue Apr 7, 2022
Lms24 added a commit that referenced this issue Apr 26, 2022
This PR fixes the AWS Lambda layer in @sentry/serverless.

The problem was caused by changes introduced in #4882.

Some unintended consequences of these changes were:

- By centralising most .npmignore files, npm-packlist did not find all the necessary (but also unnecessary) files when creating the list of files that should end up in the lambda layer Zip file. Due to the missing .npmignore files, it resorted to .gitignore which ignores generated JS files. This PR re-adds local .npmignore files in the dependencies of @sentry/serverless to fix this.

- @sentry/tracing is a special case where we need to execute prepack.ts also after building. The call to prepack was missing the --bundles flag, resulting in a weird directory structure. This PR adds the flag and adjusts the sideEffects entry we need for our integration tests.

As a hotfix, this PR ensures further that the directory structure in the Lambda layer conforms to the NODE_OPTIONS environment variable set by Sentry
@Lms24
Copy link
Member Author

Lms24 commented May 4, 2022

Hello everyone,

we just released the first v7 beta version of the SDK - 7.0.0-beta.0.
If you want to update your Sentry version to the latest beta, check out our migration guide to learn more about breaking changes and how to migrate successfully.

Looking forward to receiving your feedback!

@mlmmn
Copy link

mlmmn commented May 9, 2022

@Lms24 Does v7 improve multiple client support (when it comes to integrations + support for module federation)?

@Lms24
Copy link
Member Author

Lms24 commented May 10, 2022

Hi @mlmmn
Our main goals for v7 were to refactor and restructure multiple aspects of our SDK to reduce bundle size, increase tree shaking support and prepare it for upcoming features. The changes are outlined in our updated v7 roadmap. It should be noted that we decided to cut scope from the original release plan to actually get the new major version released with a clear focus and in a timely manner. To have a look at what is already implemented (most if not all breaking changes), take a look at the changelog of our latest beta version and the linked migration guide.

This overall means that multi-hub/client support was not improved greatly with v7 but rest assured that we're aware of the issues and we're working on improving this. On a side note, a very small bugfix (#5046) that fixed some occasions of transactions not being sent to the correct DSN in a multi-hub scenario recently made it into our 7.x branch. This will soon be released with the next beta.

@belgattitude
Copy link
Contributor

Worth to mention: #3865 (comment)

@AbhiPrasad
Copy link
Member

v7 has been released! Please reach out if you have any feedback!

https://github.com/getsentry/sentry-javascript/releases/tag/7.0.0

https://www.npmjs.com/package/@sentry/browser/v/7.0.0

https://www.npmjs.com/package/@sentry/node/v/7.0.0

@AbhiPrasad AbhiPrasad unpinned this issue May 30, 2022
@belgattitude
Copy link
Contributor

Congrats for the amazing 7 馃挴

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

Successfully merging a pull request may close this issue.

5 participants