search

getsentry / sentry-dart

Sentry SDK for Dart and Flutter
https://sentry.io/for/flutter/
MIT License
766 stars 239 forks source link

ci: comment high risk only once #2414

Closed buenaflor closed 2 weeks ago

buenaflor commented 2 weeks ago

closes #2413 all the duplicate comments are only noise

Tested this by checking if only one comment exists on multiple commits

skip-changelog

github-actions[bot] commented 2 weeks ago

🚨 Detected changes in high risk code 🚨

High-risk code has higher potential to break the SDK and may be hard to test. To prevent severe bugs, apply the rollout process for releasing such changes and be extra careful when changing and reviewing these files:

codecov[bot] commented 2 weeks ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 85.95%. Comparing base (bd75526) to head (907f68a).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #2414 +/- ## ========================================== + Coverage 85.06% 85.95% +0.89% ========================================== Files 257 81 -176 Lines 9191 2869 -6322 ========================================== - Hits 7818 2466 -5352 + Misses 1373 403 -970 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

github-actions[bot] commented 2 weeks ago

Android Performance metrics :rocket:

  Plain With Sentry Diff
Startup time 551.35 ms 567.26 ms 15.91 ms
Size 6.49 MiB 7.56 MiB 1.07 MiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
2966d88d828c6e2604b15b95f0d0d958acc2cc68 365.71 ms 422.50 ms 56.79 ms
affcf078275df70b443b3150a5ae60951bfa5faa 446.19 ms 476.32 ms 30.13 ms
fcd1ee46299294dbd1f2040a50690d1b37e7c1d6 298.96 ms 376.04 ms 77.09 ms
b47809a5f032149d70c9c8cdc226a1b71189ae88 333.42 ms 368.36 ms 34.95 ms
c1bb00f9a9935da3f990e1bbef4469c6bfe843f2 303.77 ms 371.88 ms 68.11 ms
f770c4cf27cf7af52784b27085d7efce09f70004 385.88 ms 449.86 ms 63.98 ms
3d305b96e5469a711cd2c67191b28bbcd997ca41 403.55 ms 469.76 ms 66.20 ms
ea60f1069d6f61b271993984e324dd2bb104e580 456.23 ms 494.48 ms 38.25 ms
8609bd8dd7ea572e5d241a59643c7570e5621bda 359.76 ms 437.40 ms 77.64 ms
7748b7cce475ff9e094eafe66ba7e3850ef59f99 356.37 ms 426.70 ms 70.33 ms

App size

Revision Plain With Sentry Diff
2966d88d828c6e2604b15b95f0d0d958acc2cc68 6.33 MiB 7.27 MiB 954.11 KiB
affcf078275df70b443b3150a5ae60951bfa5faa 6.52 MiB 7.59 MiB 1.06 MiB
fcd1ee46299294dbd1f2040a50690d1b37e7c1d6 6.16 MiB 7.13 MiB 1000.85 KiB
b47809a5f032149d70c9c8cdc226a1b71189ae88 5.94 MiB 6.96 MiB 1.02 MiB
c1bb00f9a9935da3f990e1bbef4469c6bfe843f2 6.06 MiB 7.09 MiB 1.03 MiB
f770c4cf27cf7af52784b27085d7efce09f70004 6.33 MiB 7.26 MiB 950.37 KiB
3d305b96e5469a711cd2c67191b28bbcd997ca41 6.35 MiB 7.35 MiB 1021.16 KiB
ea60f1069d6f61b271993984e324dd2bb104e580 6.49 MiB 7.55 MiB 1.07 MiB
8609bd8dd7ea572e5d241a59643c7570e5621bda 6.27 MiB 7.20 MiB 959.07 KiB
7748b7cce475ff9e094eafe66ba7e3850ef59f99 6.27 MiB 7.20 MiB 959.09 KiB

Previous results on branch: ci/comment-once-high-risk

Startup times

Revision Plain With Sentry Diff
907f68aeb482d5d8dbd8b37f56457a63d1644515 463.20 ms 496.96 ms 33.75 ms
6a513a5829ec492814436823c5dd9333e2b65309 451.60 ms 499.08 ms 47.48 ms

App size

Revision Plain With Sentry Diff
907f68aeb482d5d8dbd8b37f56457a63d1644515 6.49 MiB 7.56 MiB 1.07 MiB
6a513a5829ec492814436823c5dd9333e2b65309 6.49 MiB 7.56 MiB 1.07 MiB
github-actions[bot] commented 2 weeks ago

iOS Performance metrics :rocket:

  Plain With Sentry Diff
Startup time 1227.82 ms 1246.54 ms 18.73 ms
Size 8.38 MiB 9.76 MiB 1.39 MiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
f0fcbe1432e5c9ffab835fe700c5c061fb52aa98 1238.08 ms 1243.44 ms 5.36 ms
0a23f9840336da9128fd8f15f1a373563820df64 1252.98 ms 1276.76 ms 23.78 ms
c9784770c2d501233c3643f98ae182e51f08f6c2 1228.43 ms 1249.17 ms 20.74 ms
a40bb7c712c6ca285e8712782b99c6ec1e1d6d03 1222.75 ms 1240.88 ms 18.13 ms
6034b0a5af03e6891b1153ba6a88f095cbd7a948 1244.89 ms 1270.22 ms 25.33 ms
ef2f368c1d563e58bcb34b3198f93eb75fa71602 1259.12 ms 1277.04 ms 17.92 ms
ecb4003e3aa60b9295e74bd20dc987dafe567263 1266.94 ms 1290.24 ms 23.31 ms
e0ba81fd379b8bac05a707e1984ed1582520a584 1240.27 ms 1260.15 ms 19.88 ms
d5fb969ad284346622b7026449ed98215e728feb 1228.79 ms 1256.17 ms 27.38 ms
7ec9238637dbe38b6170da172bc0650eaa157885 1259.69 ms 1281.59 ms 21.90 ms

App size

Revision Plain With Sentry Diff
f0fcbe1432e5c9ffab835fe700c5c061fb52aa98 8.29 MiB 9.38 MiB 1.09 MiB
0a23f9840336da9128fd8f15f1a373563820df64 8.10 MiB 9.18 MiB 1.08 MiB
c9784770c2d501233c3643f98ae182e51f08f6c2 8.32 MiB 9.50 MiB 1.18 MiB
a40bb7c712c6ca285e8712782b99c6ec1e1d6d03 8.38 MiB 9.73 MiB 1.35 MiB
6034b0a5af03e6891b1153ba6a88f095cbd7a948 8.33 MiB 9.40 MiB 1.07 MiB
ef2f368c1d563e58bcb34b3198f93eb75fa71602 8.15 MiB 9.10 MiB 965.24 KiB
ecb4003e3aa60b9295e74bd20dc987dafe567263 8.09 MiB 9.07 MiB 1000.82 KiB
e0ba81fd379b8bac05a707e1984ed1582520a584 8.38 MiB 9.71 MiB 1.34 MiB
d5fb969ad284346622b7026449ed98215e728feb 8.33 MiB 9.64 MiB 1.31 MiB
7ec9238637dbe38b6170da172bc0650eaa157885 8.34 MiB 9.65 MiB 1.31 MiB

Previous results on branch: ci/comment-once-high-risk

Startup times

Revision Plain With Sentry Diff
6a513a5829ec492814436823c5dd9333e2b65309 1254.04 ms 1276.29 ms 22.24 ms
907f68aeb482d5d8dbd8b37f56457a63d1644515 1243.10 ms 1268.17 ms 25.07 ms

App size

Revision Plain With Sentry Diff
6a513a5829ec492814436823c5dd9333e2b65309 8.38 MiB 9.76 MiB 1.39 MiB
907f68aeb482d5d8dbd8b37f56457a63d1644515 8.38 MiB 9.76 MiB 1.39 MiB