Skip to content

Conversation

@supervacuus
Copy link
Collaborator

📜 Description

Aligns with the permission requirements here: getsentry/craft#722

💡 Motivation and Context

Fixes the currently failing workflow.

💚 How did you test it?

With this PR.

📝 Checklist

  • I added GH Issue ID & Linear ID
  • I added tests to verify the changes.
  • No new PII added or SDK only sends newly added PII if sendDefaultPII is enabled.
  • I updated the docs if needed.
  • I updated the wizard if needed.
  • Review from the native team if needed.
  • No breaking change or entry added to the changelog.
  • No breaking change for hybrid SDKs or communicated to hybrid SDKs.

🔮 Next steps

Which aligns with the permission requirements here: getsentry/craft#722
@github-actions
Copy link
Contributor

github-actions bot commented Jan 22, 2026

Semver Impact of This PR

🟢 Patch (bug fixes)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).


Internal Changes 🔧

  • (ci) Write permission for statuses in changelog preview by supervacuus in #5053

🤖 This preview updates automatically when you update the PR.

@github-actions
Copy link
Contributor

github-actions bot commented Jan 22, 2026

Performance metrics 🚀

  Plain With Sentry Diff
Startup time 316.24 ms 375.29 ms 59.05 ms
Size 1.58 MiB 2.19 MiB 620.21 KiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
ae7fed0 293.84 ms 380.22 ms 86.38 ms
bbc35bb 324.88 ms 425.73 ms 100.85 ms
d15471f 302.62 ms 353.84 ms 51.22 ms
27d7cf8 306.76 ms 366.66 ms 59.90 ms
27d7cf8 397.90 ms 498.65 ms 100.75 ms
3699cd5 423.60 ms 495.52 ms 71.92 ms
951caf7 323.66 ms 392.82 ms 69.16 ms
fc5ccaf 270.49 ms 363.90 ms 93.41 ms
fcec2f2 314.96 ms 373.66 ms 58.70 ms
dba088c 320.59 ms 361.29 ms 40.70 ms

App size

Revision Plain With Sentry Diff
ae7fed0 1.58 MiB 2.12 MiB 551.77 KiB
bbc35bb 1.58 MiB 2.12 MiB 553.01 KiB
d15471f 1.58 MiB 2.13 MiB 559.54 KiB
27d7cf8 1.58 MiB 2.12 MiB 549.42 KiB
27d7cf8 1.58 MiB 2.12 MiB 549.42 KiB
3699cd5 1.58 MiB 2.10 MiB 533.45 KiB
951caf7 1.58 MiB 2.13 MiB 558.77 KiB
fc5ccaf 1.58 MiB 2.13 MiB 557.54 KiB
fcec2f2 1.58 MiB 2.12 MiB 551.50 KiB
dba088c 1.58 MiB 2.13 MiB 558.99 KiB

Copy link
Member

@romtsn romtsn left a comment

Choose a reason for hiding this comment

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

thanks!

@supervacuus supervacuus merged commit 72020f8 into main Jan 22, 2026
62 of 63 checks passed
@supervacuus supervacuus deleted the supervacuus-patch-1 branch January 22, 2026 20:02
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.

3 participants