Skip to content

Fix: automated benchmark testing via Job Summary#222

Merged
EmmasBox merged 1 commit into
Mainframe-Renewal-Project:devfrom
yinjie-z:fix/benchmark-table
Mar 15, 2026
Merged

Fix: automated benchmark testing via Job Summary#222
EmmasBox merged 1 commit into
Mainframe-Renewal-Project:devfrom
yinjie-z:fix/benchmark-table

Conversation

@yinjie-z

@yinjie-z yinjie-z commented Mar 15, 2026

Copy link
Copy Markdown
Collaborator

💡 Issue Reference

Issue: #217

💻 What does this address?

I found that pull_request_target triggers often only take full effect once the workflow is merged into the repository's default branch (main). Otherwise, using a bot to post comments on PRs from forks frequently hits permission blocks (403 Forbidden).

Here's the doc :

This event runs in the context of the default branch of the base repository, rather than in the context of the merge commit, as the pull_request event does.

📟 Implementation Details

  • Uses GITHUB_STEP_SUMMARY to display performance data.
  • Bypasses the 403 Forbidden issues typically found when bots try to comment on PRs from forks.
  • Provides a clean table showing Execution Time and Memory Growth.

@github-actions github-actions Bot added the build Related to packaging, compiling, and GitHub actions label Mar 15, 2026
@yinjie-z

yinjie-z commented Mar 15, 2026

Copy link
Copy Markdown
Collaborator Author

To see the table results :

  1. Go to the Checks tab of this PR.
  2. Click on Benchmark Test in the sidebar.
  3. Select Summary to view the performance table (Avg Time & Memory Growth).

OR

  1. Scroll down to the Checks area at the bottom of the PR.
  2. Click View Details next to Benchmark Test job
  3. Select Summary in the sidebar to view the table

Example in Job Summary

@EmmasBox EmmasBox merged commit 403a5bd into Mainframe-Renewal-Project:dev Mar 15, 2026
7 checks passed
@yinjie-z yinjie-z deleted the fix/benchmark-table branch March 23, 2026 22:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build Related to packaging, compiling, and GitHub actions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants