-
-
Notifications
You must be signed in to change notification settings - Fork 19
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(sidecar): invalid semver range for source-map #667
fix(sidecar): invalid semver range for source-map #667
Conversation
@belgattitude is attempting to deploy a commit to the Sentry Team on Vercel. A member of the Team first needs to authorize it. |
Heya, thanks for the report and the fix! I'll see why there are so many changes in the lock file and approve, merge, release. |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #667 +/- ##
=======================================
Coverage 31.29% 31.29%
=======================================
Files 92 92
Lines 5435 5435
Branches 100 100
=======================================
Hits 1701 1701
Misses 3734 3734 ☔ View full report in Codecov by Sentry. |
This PR was opened by the [Changesets release](https://github.com/changesets/action) GitHub action. When you're ready to do a release, you can merge this and publish to npm yourself or [setup this action to publish automatically](https://github.com/changesets/action#with-publishing). If you're not ready to do a release yet, that's fine, whenever you add more changesets to main, this PR will be updated. # Releases ## @spotlightjs/[email protected] ### Patch Changes - Updated dependencies \[]: - @spotlightjs/[email protected] ## @spotlightjs/[email protected] ### Patch Changes - Updated dependencies \[[`03e386af2024eda44c02952a06433e4d4ecb3356`](03e386a), [`416fecd778d4829d490b07ec4d500946fa439210`](416fecd)]: - @spotlightjs/[email protected] - @spotlightjs/[email protected] ## @spotlightjs/[email protected] ### Patch Changes - fix: Always try to use sidecar along with direct SDK transport ([#672](#672)) ## @spotlightjs/[email protected] ### Patch Changes - Fix invalid semver range for source-map dependency ([#667](#667)) ## @spotlightjs/[email protected] ### Patch Changes - Updated dependencies \[[`03e386af2024eda44c02952a06433e4d4ecb3356`](03e386a), [`416fecd778d4829d490b07ec4d500946fa439210`](416fecd)]: - @spotlightjs/[email protected] - @spotlightjs/[email protected] Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Before opening this PR:
pnpm changeset:add
Issue
The range specified isn't technically correct (pnpm accepts this though). Check with semver
The fix is simple:
Workaround till fixed upstream