Skip to content

Conversation

@itaybre
Copy link
Contributor

@itaybre itaybre commented Nov 7, 2025

Looks like GH actions events may be missing the PR labels.

This PR fetch them manually instead of depending on the obtained data

Closes #6722

@itaybre itaybre added the ready-to-merge Use this label to trigger all PR workflows label Nov 7, 2025
@codecov
Copy link

codecov bot commented Nov 7, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.125%. Comparing base (3a3d93b) to head (313832c).
⚠️ Report is 19 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files

Impacted file tree graph

@@              Coverage Diff              @@
##              main     #6721       +/-   ##
=============================================
- Coverage   85.163%   85.125%   -0.038%     
=============================================
  Files          453       453               
  Lines        27689     27706       +17     
  Branches     12111     12143       +32     
=============================================
+ Hits         23581     23585        +4     
- Misses        4063      4076       +13     
  Partials        45        45               

see 64 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 3a3d93b...313832c. Read the comment docs.

@github-actions
Copy link
Contributor

github-actions bot commented Nov 7, 2025

Performance metrics 🚀

  Plain With Sentry Diff
Startup time 1224.21 ms 1254.89 ms 30.68 ms
Size 24.15 KiB 1.01 MiB 1015.23 KiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
2481950 1221.04 ms 1248.98 ms 27.94 ms
9766cb1 1233.54 ms 1267.65 ms 34.10 ms
7eafbde 1212.16 ms 1243.64 ms 31.48 ms
01faa71 1238.81 ms 1263.98 ms 25.17 ms
d0f70ce 1226.54 ms 1247.04 ms 20.50 ms
7f2f69c 1237.61 ms 1266.96 ms 29.35 ms
fc0757d 1231.83 ms 1248.98 ms 17.15 ms
5cbd333 1220.78 ms 1234.15 ms 13.36 ms
e0e5391 1191.21 ms 1243.14 ms 51.93 ms
f92cfa9 1228.45 ms 1251.33 ms 22.88 ms

App size

Revision Plain With Sentry Diff
2481950 23.74 KiB 872.74 KiB 849.00 KiB
9766cb1 23.75 KiB 1023.78 KiB 1000.03 KiB
7eafbde 23.75 KiB 927.53 KiB 903.78 KiB
01faa71 23.75 KiB 926.77 KiB 903.02 KiB
d0f70ce 23.75 KiB 913.09 KiB 889.34 KiB
7f2f69c 23.75 KiB 913.38 KiB 889.63 KiB
fc0757d 23.75 KiB 850.73 KiB 826.98 KiB
5cbd333 23.74 KiB 969.66 KiB 945.92 KiB
e0e5391 23.74 KiB 1022.21 KiB 998.46 KiB
f92cfa9 23.75 KiB 855.38 KiB 831.62 KiB

Previous results on branch: itay/fetch_pr_labels

Startup times

Revision Plain With Sentry Diff
46e2bea 1192.58 ms 1219.68 ms 27.10 ms

App size

Revision Plain With Sentry Diff
46e2bea 23.75 KiB 1.00 MiB 1001.52 KiB

Copy link
Member

@philprime philprime left a comment

Choose a reason for hiding this comment

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

LGTM

@itaybre itaybre merged commit 543b791 into main Nov 12, 2025
187 of 194 checks passed
@itaybre itaybre deleted the itay/fetch_pr_labels branch November 12, 2025 12:57
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.

chore: Update workflow to fetch labels using github API

3 participants