Skip to content

refactor: enhance benchmark workflow with improved caching and error handling#455

Merged
ReneWerner87 merged 3 commits intomasterfrom
bench_workflow
Feb 7, 2026
Merged

refactor: enhance benchmark workflow with improved caching and error handling#455
ReneWerner87 merged 3 commits intomasterfrom
bench_workflow

Conversation

@ReneWerner87
Copy link
Member

No description provided.

@ReneWerner87 ReneWerner87 requested a review from a team as a code owner February 7, 2026 20:56
@ReneWerner87 ReneWerner87 requested review from Copilot, efectn, gaby and sixcolors and removed request for a team February 7, 2026 20:56
@gemini-code-assist
Copy link

Note

Gemini is unable to generate a summary for this pull request due to the file types involved not being currently supported.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR refactors the GitHub Actions benchmark workflow to better support PR benchmarking by improving trigger configuration, adding stricter bash error handling, and introducing cache-based comparisons against the default branch.

Changes:

  • Add workflow_dispatch, tighten paths globs, and adjust job/permissions for PR commenting.
  • Make benchmark execution more robust (set -euo pipefail) and avoid directory state leakage via subshells.
  • Introduce restore/save caching and split benchmark-action usage into PR comparison vs default-branch publishing.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@ReneWerner87 ReneWerner87 merged commit 65a4796 into master Feb 7, 2026
9 checks passed
@ReneWerner87 ReneWerner87 deleted the bench_workflow branch February 7, 2026 21:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants