Skip to content

Conversation

@itaybre
Copy link
Contributor

@itaybre itaybre commented Oct 29, 2025

Test runners on Cirrus Runners

Still pending:

  • Merge test workflow with the regular test workflow
  • Run Cirrus runners only on main branch

#skip-changelog

@github-actions
Copy link
Contributor

github-actions bot commented Oct 29, 2025

Messages
📖 Do not forget to update Sentry-docs with your feature once the pull request gets approved.

Generated by 🚫 dangerJS against d30c43f

@codecov
Copy link

codecov bot commented Oct 29, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.181%. Comparing base (52bdc10) to head (d30c43f).
✅ All tests successful. No failed tests found.

Additional details and impacted files

Impacted file tree graph

@@              Coverage Diff              @@
##              main     #6603       +/-   ##
=============================================
+ Coverage   85.149%   85.181%   +0.032%     
=============================================
  Files          453       453               
  Lines        27689     27689               
  Branches     12115     12115               
=============================================
+ Hits         23577     23586        +9     
+ Misses        4066      4058        -8     
+ Partials        46        45        -1     

see 5 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 52bdc10...d30c43f. Read the comment docs.

@github-actions
Copy link
Contributor

github-actions bot commented Oct 30, 2025

Performance metrics 🚀

  Plain With Sentry Diff
Startup time 1223.20 ms 1255.20 ms 32.00 ms
Size 23.75 KiB 1022.78 KiB 999.04 KiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
8b966fc 1220.52 ms 1253.15 ms 32.63 ms
ebc3a34 1236.24 ms 1261.02 ms 24.78 ms
08fc5f9 1237.84 ms 1270.06 ms 32.23 ms
7f2f69c 1237.61 ms 1266.96 ms 29.35 ms
07f6759 1232.98 ms 1267.59 ms 34.61 ms
7d23639 1237.93 ms 1243.04 ms 5.11 ms
4e3915a 1230.02 ms 1258.90 ms 28.88 ms
7416ffc 1225.55 ms 1241.80 ms 16.25 ms
f5d75da 1230.57 ms 1264.30 ms 33.74 ms
b5a7583 1238.22 ms 1263.94 ms 25.71 ms

App size

Revision Plain With Sentry Diff
8b966fc 23.75 KiB 1.02 MiB 1019.10 KiB
ebc3a34 23.75 KiB 919.91 KiB 896.16 KiB
08fc5f9 23.75 KiB 995.43 KiB 971.68 KiB
7f2f69c 23.75 KiB 913.38 KiB 889.63 KiB
07f6759 23.75 KiB 997.26 KiB 973.52 KiB
7d23639 23.75 KiB 891.01 KiB 867.26 KiB
4e3915a 23.75 KiB 858.69 KiB 834.94 KiB
7416ffc 23.75 KiB 913.63 KiB 889.88 KiB
f5d75da 23.75 KiB 1004.80 KiB 981.05 KiB
b5a7583 23.75 KiB 913.44 KiB 889.68 KiB

Previous results on branch: itay/cirrus_labs_tests

Startup times

Revision Plain With Sentry Diff
c8bde6b 1226.34 ms 1259.82 ms 33.48 ms
13d1e64 1224.84 ms 1259.24 ms 34.40 ms
5cb95b6 1202.55 ms 1230.11 ms 27.56 ms
94a4564 1230.30 ms 1266.62 ms 36.32 ms
f17adae 1221.86 ms 1251.98 ms 30.12 ms
527a1ba 1233.75 ms 1258.78 ms 25.03 ms
31bcdb9 1219.98 ms 1260.59 ms 40.61 ms
57d91eb 1227.60 ms 1260.54 ms 32.94 ms

App size

Revision Plain With Sentry Diff
c8bde6b 23.75 KiB 1023.82 KiB 1000.07 KiB
13d1e64 23.75 KiB 1023.82 KiB 1000.07 KiB
5cb95b6 23.75 KiB 1023.82 KiB 1000.07 KiB
94a4564 23.75 KiB 1.02 MiB 1016.67 KiB
f17adae 23.75 KiB 1023.81 KiB 1000.07 KiB
527a1ba 23.75 KiB 1023.81 KiB 1000.06 KiB
31bcdb9 23.75 KiB 1023.82 KiB 1000.07 KiB
57d91eb 23.75 KiB 1023.82 KiB 1000.07 KiB

@github-actions
Copy link
Contributor

github-actions bot commented Nov 6, 2025

🚨 Detected changes in high risk code 🚨

High-risk code can easily blow up and is hard to test. We had severe bugs in the past. Be extra careful when changing these files, and have an extra careful look at these:

  • .github/file-filters.yml

@itaybre itaybre added the ready-to-merge Use this label to trigger all PR workflows label Nov 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-to-merge Use this label to trigger all PR workflows

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants