Skip to content

fix benchmark result posting#159

Draft
hakkelt wants to merge 1 commit intoJuliaFirstOrder:masterfrom
hakkelt:maybe-fix-benchmarking
Draft

fix benchmark result posting#159
hakkelt wants to merge 1 commit intoJuliaFirstOrder:masterfrom
hakkelt:maybe-fix-benchmarking

Conversation

@hakkelt
Copy link
Copy Markdown
Contributor

@hakkelt hakkelt commented Apr 15, 2026

In this PR, I try to fix this problem by dividing the benchmarking into two phases:

  1. Run the benchmark, save the results to a file, and upload it as an artifact. -> It does not need any special permissions.
  2. Download the artifact and post the content as a comment. -> This gets triggered by the first phase via "workflow_run", and my hope is that it is possible to selectively set write permission to this job only.

@hakkelt hakkelt force-pushed the maybe-fix-benchmarking branch from 517d2b2 to ab571a6 Compare April 15, 2026 22:08
@codecov
Copy link
Copy Markdown

codecov bot commented Apr 15, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.78%. Comparing base (83c73c9) to head (ab571a6).

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #159      +/-   ##
==========================================
- Coverage   94.79%   94.78%   -0.01%     
==========================================
  Files          79       79              
  Lines        2498     2494       -4     
==========================================
- Hits         2368     2364       -4     
  Misses        130      130              

☔ 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.

@hakkelt
Copy link
Copy Markdown
Contributor Author

hakkelt commented Apr 15, 2026

To test this PR, post_benchmark_comment.yml needs to be on master already... So I'll play around a bit in some playground repositories, then I'll report if it works.

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.

1 participant