Skip to content

Conversation

@bruno-garcia
Copy link
Member

See:

getsentry/sentry-dart#433 (comment)

Seems to me we could simplify the instructions about uploading symbols which now is broken down by headings titled after OS's. But I tried to keep it simple for now, just removing the note that is not supported

@vercel
Copy link

vercel bot commented Feb 2, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
sentry-docs ❌ Failed (Inspect) Feb 19, 2025 6:41pm
2 Skipped Deployments
Name Status Preview Comments Updated (UTC)
changelog ⬜️ Ignored (Inspect) Visit Preview Feb 19, 2025 6:41pm
develop-docs ⬜️ Ignored (Inspect) Visit Preview Feb 19, 2025 6:41pm

@codecov
Copy link

codecov bot commented Feb 2, 2025

Bundle Report

Changes will increase total bundle size by 100.96kB (0.49%) ⬆️. This is within the configured threshold ✅

Detailed changes
Bundle name Size Change
sentry-docs-server-cjs 10.8MB 100.97kB (0.94%) ⬆️
sentry-docs-client-array-push 9.38MB -6 bytes (-0.0%) ⬇️

Affected Assets, Files, and Routes:

view changes for bundle: sentry-docs-server-cjs

Assets Changed:

Asset Name Size Change Total Size Change (%)
1729.js 33.65kB 1.67MB 2.06%
../instrumentation.js 33.65kB 998.88kB 3.49%
9523.js 33.68kB 975.12kB 3.58%
view changes for bundle: sentry-docs-client-array-push

Assets Changed:

Asset Name Size Change Total Size Change (%)
static/chunks/pages/_app-*.js -3 bytes 866.04kB -0.0%
static/chunks/9557-*.js -3 bytes 391.11kB -0.0%
static/8e2jFLvEvdpvNI8wYS5B2/_buildManifest.js (New) 578 bytes 578 bytes 100.0% 🚀
static/8e2jFLvEvdpvNI8wYS5B2/_ssgManifest.js (New) 77 bytes 77 bytes 100.0% 🚀
static/MOoMWxn6DqnCZgGx1naJd/_buildManifest.js (Deleted) -578 bytes 0 bytes -100.0% 🗑️
static/MOoMWxn6DqnCZgGx1naJd/_ssgManifest.js (Deleted) -77 bytes 0 bytes -100.0% 🗑️

Copy link
Contributor

@vaind vaind left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the change, LGTM. FYI: there was another section (line 13) I've removed on your branch.

@bruno-garcia
Copy link
Member Author

[codecov] Sending telemetry data on issues and performance to Codecov. To disable telemetry, set `options.telemetry` to `false`.
[@sentry/nextjs - Edge] Info: Sending telemetry data on issues and performance to Sentry. To disable telemetry, set `options.telemetry` to `false`.
[@sentry/nextjs - Edge] Info: Successfully uploaded source maps to Sentry
<w> [webpack.cache.PackFileCacheStrategy] Serializing big strings (120kiB) impacts deserialization performance (consider using Buffer instead and decode when needed)
<w> [webpack.cache.PackFileCacheStrategy] Serializing big strings (242kiB) impacts deserialization performance (consider using Buffer instead and decode when needed)
[codecov] Sending telemetry data on issues and performance to Codecov. To disable telemetry, set `options.telemetry` to `false`.
[@sentry/nextjs - Client] Info: Sending telemetry data on issues and performance to Sentry. To disable telemetry, set `options.telemetry` to `false`.
[@sentry/nextjs - Client] Info: Successfully uploaded source maps to Sentry
Error: Failed to interpolate expression: "packages.checksum('sentry.javascript.browser', 'debug.min.js', 'sha384-base64')"
at <unknown> (.next/server/chunks/9567.js:2:15523)
at l (.next/server/chunks/9567.js:2:14908)
at <unknown> (.next/server/chunks/9567.js:2:15086)
at l (.next/server/chunks/4899.js:1:1033774)
at node (code) (.next/server/chunks/4899.js:1:1034460)
at node (mdxJsxFlowElement<CodeBlock>) (.next/server/chunks/4899.js:1:1034649)
at node (mdxJsxFlowElement<CodeTabs>) (.next/server/chunks/4899.js:1:1034649)
at node (element) (.next/server/chunks/4899.js:1:1034649)
at node (root) (.next/server/chunks/4899.js:1:1034649)
at u (.next/server/chunks/4899.js:1:1034727)
⨯ Static worker exited with code: 1 and signal: null
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
Error: Command "yarn run build" exited with 1

flakey ci @coolguyzone ?

auto-merge was automatically disabled February 20, 2025 18:35

Pull request was closed

@bruno-garcia bruno-garcia reopened this Feb 20, 2025
@bruno-garcia bruno-garcia enabled auto-merge (squash) February 20, 2025 18:36
@coolguyzone
Copy link
Contributor

@bruno-garcia try rebasing from the latest master branch, that should resolve this.

@bruno-garcia
Copy link
Member Author

@buenaflor do u mind taking this over? feel free to close my PR

@buenaflor buenaflor closed this Apr 11, 2025
auto-merge was automatically disabled April 11, 2025 09:20

Pull request was closed

@github-actions github-actions bot locked and limited conversation to collaborators Apr 27, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants