Skip to content

Conversation

antonis
Copy link
Contributor

@antonis antonis commented Sep 29, 2025

📢 Type of change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring

📜 Description

This PR fixes the breaking change in https://github.com/getsentry/github-workflows/releases/tag/3.0.0 and bumps workflows to v3

💡 Motivation and Context

Breaking changes

💚 How did you test it?

CI (job run)

📝 Checklist

  • I added tests to verify 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.
  • All tests passing
  • No breaking changes

🔮 Next steps

#skip-changelog

Copy link
Contributor

github-actions bot commented Sep 29, 2025

Android (new) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 346.21 ms 387.00 ms 40.79 ms
Size 7.15 MiB 8.43 MiB 1.28 MiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
21c9e75+dirty 356.73 ms 381.06 ms 24.33 ms
a0b15d6+dirty 414.33 ms 448.85 ms 34.52 ms
77061ed+dirty 359.40 ms 424.10 ms 64.70 ms
818a608+dirty 350.29 ms 397.38 ms 47.09 ms
b3b5b0d+dirty 361.42 ms 403.90 ms 42.48 ms
1853710+dirty 360.67 ms 396.28 ms 35.61 ms
a2bb688+dirty 371.19 ms 389.18 ms 17.99 ms
20daa0a+dirty 352.33 ms 424.30 ms 71.98 ms
a02e30b+dirty 346.13 ms 381.76 ms 35.62 ms
e2fa43d+dirty 326.56 ms 372.88 ms 46.32 ms

App size

Revision Plain With Sentry Diff
21c9e75+dirty 7.15 MiB 8.42 MiB 1.27 MiB
a0b15d6+dirty 7.15 MiB 8.42 MiB 1.27 MiB
77061ed+dirty 7.15 MiB 8.41 MiB 1.26 MiB
818a608+dirty 7.15 MiB 8.41 MiB 1.26 MiB
b3b5b0d+dirty 7.15 MiB 8.41 MiB 1.26 MiB
1853710+dirty 7.15 MiB 8.41 MiB 1.26 MiB
a2bb688+dirty 7.15 MiB 8.43 MiB 1.28 MiB
20daa0a+dirty 7.15 MiB 8.42 MiB 1.27 MiB
a02e30b+dirty 7.15 MiB 8.42 MiB 1.27 MiB
e2fa43d+dirty 7.15 MiB 8.42 MiB 1.27 MiB

Previous results on branch: antonis/fix-worflows-for-v3

Startup times

Revision Plain With Sentry Diff
ebd5174+dirty 404.26 ms 462.48 ms 58.22 ms

App size

Revision Plain With Sentry Diff
ebd5174+dirty 7.15 MiB 8.43 MiB 1.28 MiB

Copy link
Contributor

github-actions bot commented Sep 29, 2025

iOS (legacy) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 1234.73 ms 1238.57 ms 3.84 ms
Size 2.63 MiB 3.99 MiB 1.36 MiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
1bea095+dirty 1234.14 ms 1233.96 ms -0.18 ms
20daa0a+dirty 1233.12 ms 1233.35 ms 0.22 ms
ba75c7c+dirty 1235.86 ms 1226.45 ms -9.41 ms
6479fd5+dirty 1228.48 ms 1231.32 ms 2.84 ms
7480abe+dirty 1220.53 ms 1244.18 ms 23.65 ms
c4e097a+dirty 1229.09 ms 1242.61 ms 13.53 ms
69602ce+dirty 1235.65 ms 1230.82 ms -4.83 ms
d916aa3+dirty 1221.02 ms 1228.98 ms 7.96 ms
d751a5d+dirty 1215.57 ms 1220.56 ms 4.99 ms
955f2eb+dirty 1235.06 ms 1253.88 ms 18.81 ms

App size

Revision Plain With Sentry Diff
1bea095+dirty 2.63 MiB 3.99 MiB 1.35 MiB
20daa0a+dirty 2.63 MiB 3.81 MiB 1.18 MiB
ba75c7c+dirty 2.63 MiB 3.81 MiB 1.18 MiB
6479fd5+dirty 2.63 MiB 3.96 MiB 1.33 MiB
7480abe+dirty 2.63 MiB 3.96 MiB 1.33 MiB
c4e097a+dirty 2.63 MiB 3.87 MiB 1.24 MiB
69602ce+dirty 2.63 MiB 3.91 MiB 1.28 MiB
d916aa3+dirty 2.63 MiB 3.81 MiB 1.18 MiB
d751a5d+dirty 2.63 MiB 3.98 MiB 1.34 MiB
955f2eb+dirty 2.63 MiB 3.98 MiB 1.35 MiB

Previous results on branch: antonis/fix-worflows-for-v3

Startup times

Revision Plain With Sentry Diff
ebd5174+dirty 1237.90 ms 1240.60 ms 2.71 ms

App size

Revision Plain With Sentry Diff
ebd5174+dirty 2.63 MiB 3.99 MiB 1.35 MiB

Copy link
Contributor

github-actions bot commented Sep 29, 2025

iOS (new) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 1212.47 ms 1221.83 ms 9.36 ms
Size 3.19 MiB 4.56 MiB 1.37 MiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
1bea095+dirty 1218.93 ms 1212.37 ms -6.57 ms
20daa0a+dirty 1227.71 ms 1233.72 ms 6.01 ms
ba75c7c+dirty 1236.14 ms 1240.69 ms 4.55 ms
6479fd5+dirty 1207.80 ms 1215.12 ms 7.32 ms
7480abe+dirty 1219.84 ms 1223.60 ms 3.76 ms
c4e097a+dirty 1231.35 ms 1246.91 ms 15.56 ms
69602ce+dirty 1230.59 ms 1230.84 ms 0.24 ms
d916aa3+dirty 1211.02 ms 1221.33 ms 10.31 ms
d751a5d+dirty 1212.22 ms 1217.94 ms 5.71 ms
955f2eb+dirty 1225.78 ms 1239.27 ms 13.49 ms

App size

Revision Plain With Sentry Diff
1bea095+dirty 3.19 MiB 4.55 MiB 1.37 MiB
20daa0a+dirty 3.19 MiB 4.38 MiB 1.19 MiB
ba75c7c+dirty 3.19 MiB 4.38 MiB 1.19 MiB
6479fd5+dirty 3.19 MiB 4.53 MiB 1.35 MiB
7480abe+dirty 3.19 MiB 4.53 MiB 1.35 MiB
c4e097a+dirty 3.19 MiB 4.44 MiB 1.25 MiB
69602ce+dirty 3.19 MiB 4.48 MiB 1.29 MiB
d916aa3+dirty 3.19 MiB 4.38 MiB 1.19 MiB
d751a5d+dirty 3.19 MiB 4.54 MiB 1.36 MiB
955f2eb+dirty 3.19 MiB 4.55 MiB 1.36 MiB

Previous results on branch: antonis/fix-worflows-for-v3

Startup times

Revision Plain With Sentry Diff
ebd5174+dirty 1234.37 ms 1233.81 ms -0.55 ms

App size

Revision Plain With Sentry Diff
ebd5174+dirty 3.19 MiB 4.55 MiB 1.37 MiB

# Conflicts:
#	.github/workflows/danger.yml
#	.github/workflows/update-deps.yml
Copy link
Contributor

github-actions bot commented Sep 30, 2025

Android (legacy) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 452.91 ms 484.79 ms 31.88 ms
Size 17.75 MiB 19.70 MiB 1.95 MiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
77061ed+dirty 369.55 ms 408.35 ms 38.80 ms
a2bb688+dirty 409.65 ms 410.45 ms 0.80 ms
000da7a 454.46 ms 445.00 ms -9.46 ms
a0b15d6 423.06 ms 437.77 ms 14.71 ms
af9331b 449.77 ms 479.20 ms 29.43 ms
69602ce 417.47 ms 443.52 ms 26.05 ms
7480abe+dirty 411.60 ms 405.81 ms -5.78 ms
8a4ce6f 422.88 ms 408.33 ms -14.55 ms
bc9680d 375.15 ms 401.12 ms 25.97 ms
46da307 455.92 ms 443.79 ms -12.13 ms

App size

Revision Plain With Sentry Diff
77061ed+dirty 17.75 MiB 19.68 MiB 1.94 MiB
a2bb688+dirty 17.75 MiB 19.70 MiB 1.95 MiB
000da7a 17.75 MiB 19.68 MiB 1.94 MiB
a0b15d6 17.75 MiB 20.15 MiB 2.41 MiB
af9331b 17.75 MiB 19.68 MiB 1.94 MiB
69602ce 17.75 MiB 19.68 MiB 1.94 MiB
7480abe+dirty 17.75 MiB 19.68 MiB 1.94 MiB
8a4ce6f 17.75 MiB 19.68 MiB 1.94 MiB
bc9680d 17.75 MiB 20.15 MiB 2.41 MiB
46da307 17.75 MiB 19.68 MiB 1.93 MiB

Previous results on branch: antonis/fix-worflows-for-v3

Startup times

Revision Plain With Sentry Diff
ebd5174+dirty 427.46 ms 455.00 ms 27.54 ms

App size

Revision Plain With Sentry Diff
ebd5174+dirty 17.75 MiB 19.70 MiB 1.95 MiB

@antonis antonis changed the title fix(deps): Fix workflow runs breakage after bump to v3 fix(deps): Fix workflow breakaging changes and bump to v3 Oct 3, 2025
@antonis antonis changed the title fix(deps): Fix workflow breakaging changes and bump to v3 chore(deps): bump getsentry/github-workflows from 2 to 3 and fix breaking changes Oct 3, 2025
@antonis antonis added the Blocked label Oct 6, 2025
@antonis
Copy link
Contributor Author

antonis commented Oct 6, 2025

Currently blocked in getsentry/github-workflows#128

@antonis antonis added the dependencies Pull requests that update a dependency file label Oct 6, 2025
vaind added a commit that referenced this pull request Oct 9, 2025
Tests whether updater has permission issues when running in a pr branch. Related to #5218 and getsentry/github-workflows#128
@vaind vaind mentioned this pull request Oct 9, 2025
10 tasks
@antonis
Copy link
Contributor Author

antonis commented Oct 10, 2025

Unblock when:

@vaind
Copy link
Collaborator

vaind commented Oct 10, 2025

@antonis i've bumped updater to v3 (currently pointing to 3.1.0) and everything but the javascript update worked fine. That one may have been an issue with the update being merged recently and not on this branch so I've merged main in.

@antonis
Copy link
Contributor Author

antonis commented Oct 10, 2025

Thank you @vaind 🙇 All the updater jobs are 🟢 now 🎉

@antonis antonis removed the Blocked label Oct 10, 2025
@antonis antonis marked this pull request as ready for review October 10, 2025 09:08
@antonis antonis merged commit c94a927 into main Oct 10, 2025
75 checks passed
@antonis antonis deleted the antonis/fix-worflows-for-v3 branch October 10, 2025 11:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants