Skip to content

Update GH actions run vars #3013

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

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Conversation

denrase
Copy link
Collaborator

@denrase denrase commented Jun 25, 2025

#skip-changelog

📜 Description

We should not use user modifiable vars in GH run commands directly and without double quotes.

Copy link

codecov bot commented Jun 25, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 86.13%. Comparing base (bb57a41) to head (e1bef6e).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3013      +/-   ##
==========================================
- Coverage   87.72%   86.13%   -1.60%     
==========================================
  Files         286      187      -99     
  Lines        9335     5855    -3480     
==========================================
- Hits         8189     5043    -3146     
+ Misses       1146      812     -334     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@denrase denrase marked this pull request as ready for review June 25, 2025 11:06
Copy link
Contributor

github-actions bot commented Jun 25, 2025

iOS Performance metrics 🚀

  Plain With Sentry Diff
Startup time 1250.59 ms 1268.31 ms 17.72 ms
Size 7.85 MiB 9.45 MiB 1.59 MiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
547db82 1248.23 ms 1260.90 ms 12.67 ms
25fdf12 1235.94 ms 1257.83 ms 21.89 ms
04bd9e6 1230.78 ms 1250.71 ms 19.94 ms
7273303 1246.50 ms 1262.06 ms 15.56 ms
3a69405 1292.84 ms 1303.96 ms 11.12 ms
f1314d5 1246.46 ms 1270.92 ms 24.46 ms
be8cafe 1239.94 ms 1266.20 ms 26.27 ms
8d64376 1260.92 ms 1289.32 ms 28.40 ms
1e094d3 1237.54 ms 1243.51 ms 5.97 ms
ed08c68 1233.52 ms 1257.58 ms 24.06 ms

App size

Revision Plain With Sentry Diff
547db82 8.38 MiB 9.74 MiB 1.36 MiB
25fdf12 8.42 MiB 9.91 MiB 1.49 MiB
04bd9e6 8.33 MiB 9.61 MiB 1.27 MiB
7273303 8.33 MiB 9.40 MiB 1.07 MiB
3a69405 8.15 MiB 9.15 MiB 1018.56 KiB
f1314d5 8.10 MiB 9.08 MiB 1004.30 KiB
be8cafe 8.33 MiB 9.40 MiB 1.07 MiB
8d64376 8.16 MiB 9.17 MiB 1.01 MiB
1e094d3 8.29 MiB 9.37 MiB 1.08 MiB
ed08c68 8.42 MiB 9.91 MiB 1.49 MiB

Previous results on branch: chore/update-gh-actions-run-vars

Startup times

Revision Plain With Sentry Diff
3dc7afc 1258.55 ms 1274.74 ms 16.19 ms

App size

Revision Plain With Sentry Diff
3dc7afc 7.85 MiB 9.45 MiB 1.59 MiB

Copy link
Contributor

Android Performance metrics 🚀

  Plain With Sentry Diff
Startup time 532.00 ms 568.32 ms 36.32 ms
Size 6.54 MiB 7.53 MiB 1016.34 KiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
c73ab67 353.82 ms 408.71 ms 54.90 ms
e66e71e 296.84 ms 345.43 ms 48.59 ms
1c926c2 424.57 ms 493.89 ms 69.32 ms
a817b8f 383.02 ms 445.17 ms 62.15 ms
8a10ab7 436.20 ms 509.43 ms 73.23 ms
ef31c7f 311.39 ms 359.33 ms 47.94 ms
2966d88 365.71 ms 422.50 ms 56.79 ms
134c9f8 301.40 ms 352.65 ms 51.26 ms
9fe67d5 373.80 ms 444.24 ms 70.45 ms
ee0ca56 355.35 ms 421.13 ms 65.78 ms

App size

Revision Plain With Sentry Diff
c73ab67 6.15 MiB 7.13 MiB 999.97 KiB
e66e71e 6.06 MiB 7.09 MiB 1.03 MiB
1c926c2 6.46 MiB 7.48 MiB 1.03 MiB
a817b8f 6.16 MiB 7.14 MiB 1003.22 KiB
8a10ab7 6.27 MiB 7.20 MiB 958.60 KiB
ef31c7f 6.06 MiB 7.09 MiB 1.03 MiB
2966d88 6.33 MiB 7.27 MiB 954.11 KiB
134c9f8 5.94 MiB 6.95 MiB 1.01 MiB
9fe67d5 6.33 MiB 7.29 MiB 987.27 KiB
ee0ca56 6.33 MiB 7.30 MiB 992.52 KiB

@denrase denrase requested a review from buenaflor June 25, 2025 13:08
@denrase denrase requested a review from buenaflor June 26, 2025 13:28
@denrase
Copy link
Collaborator Author

denrase commented Jun 26, 2025

@buenaflor Added one more

@buenaflor
Copy link
Contributor

@denrase I think the newest commit breaks the CI

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.

2 participants