Update bench.yml

[sbryngelson]

User description

add bench for benchmarking

---

PR Type

Enhancement

---

Description

• Expand benchmark workflow trigger conditions

• Add 'wilfonba' user to authorized PR submitters

• Simplify conditional formatting in GitHub Actions

---

Diagram Walkthrough

flowchart LR
  A["GitHub PR Event"] --> B["Check Repository"]
  B --> C["Check File Changes"]
  C --> D["Check User Authorization"]
  D --> E["Run Benchmark"]
  F["sbryngelson"] --> D
  G["wilfonba"] --> D

Loading

File Walkthrough

	Relevant files
Configuration changes	bench.yml Expand benchmark workflow user authorization                          .github/workflows/bench.yml Reformatted multi-line conditional into single line Added 'wilfonba' to authorized users for benchmark triggers Removed spacing around operators for consistency +1/-4

---

[qodo-merge-pro]

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪

🧪 No relevant tests

🔒 No security concerns identified

⚡ Recommended focus areas for review Readability Issue The complex conditional logic has been compressed into a single line, making it difficult to read and maintain. The original multi-line format was more readable and easier to debug. if: ${{ github.repository=='MFlowCode/MFC' && needs.file-changes.outputs.checkall=='true' && ((github.event_name=='pull_request_review' && github.event.review.state=='approved') || (github.event_name=='pull_request' && (github.event.pull_request.user.login=='sbryngelson' || github.event.pull_request.user.login=='wilfonba'))) }} needs: file-changes

[Copilot]

Pull Request Overview

This PR updates the GitHub Actions workflow configuration for benchmarking to add an additional authorized user and reformats the conditional logic. The change allows both 'sbryngelson' and 'wilfonba' users to trigger benchmark runs on pull requests.

Key Changes

• Added 'wilfonba' as an authorized user who can trigger benchmarks via pull requests
• Reformatted the conditional logic from multi-line to single-line format

[Copilot]

[nitpick] This complex conditional logic should be split across multiple lines for better readability and maintainability. The original multi-line format was more readable than this single long line.

[qodo-merge-pro]

PR Code Suggestions ✨

Explore these optional code suggestions:

Category	Suggestion	Impact
General	Improve conditional readability formatting The complex conditional logic is difficult to read and maintain when compressed into a single line. Breaking it into multiple lines with proper indentation would improve readability and make future modifications easier. .github/workflows/bench.yml [27] -if: ${{ github.repository=='MFlowCode/MFC' && needs.file-changes.outputs.checkall=='true' && ((github.event_name=='pull_request_review' && github.event.review.state=='approved') || (github.event_name=='pull_request' && (github.event.pull_request.user.login=='sbryngelson' || github.event.pull_request.user.login=='wilfonba'))) }} +if: ${{ github.repository == 'MFlowCode/MFC' && + needs.file-changes.outputs.checkall == 'true' && + ((github.event_name == 'pull_request_review' && github.event.review.state == 'approved') || + (github.event_name == 'pull_request' && + (github.event.pull_request.user.login == 'sbryngelson' || + github.event.pull_request.user.login == 'wilfonba'))) }} Suggestion importance[1-10]: 6 __ Why: The suggestion correctly points out that the PR's change to a single-line if condition harms readability and proposes reverting to a multi-line format, which is a valid improvement for maintainability.	Low
More

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 43.67%. Comparing base (a1d1576) to head (2d1ed2e).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff           @@
##           master     #973   +/-   ##
=======================================
  Coverage   43.67%   43.67%           
=======================================
  Files          70       70           
  Lines       19818    19818           
  Branches     2473     2473           
=======================================
  Hits         8655     8655           
  Misses       9644     9644           
  Partials     1519     1519           

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