Skip to content

chore: setup benchmark compare - #24

Merged
bengl merged 1 commit into
nodejs:mainfrom
legendecas:bench-summary
Nov 2, 2026
Merged

bengl merged 1 commit into
nodejs:mainfrom
legendecas:bench-summary

Conversation

@legendecas

@legendecas legendecas commented Nov 1, 2026 •

Copy link
Copy Markdown
Member

Run the benchmark and compare a PR with the main (PR base) branch. The result will be shown on the GHA summary page as a reference.

@bengl bengl left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Hmm, how should this work withthe pre-existing benchmark job in ci.yml? Maybe that one should run only on non-PR builds?

@legendecas

legendecas commented Nov 1, 2026 •

Copy link
Copy Markdown
Member Author

The pre-existing benchmark job will be the "push" event in the benchmark workflow, simply running the benchmark scripts on each push: https://github.com/nodejs/microtel/pull/24/changes#diff-aacbd0338bcf598920c120b69adaba2672acc7ea314cfca0f792bebe5fe2acd0R25-R28

@bengl

bengl commented Nov 1, 2026

Copy link
Copy Markdown
Member

Alright SGTM.

@bengl
bengl merged commit 31122f8 into nodejs:main Nov 2, 2026
3 checks passed
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.

2 participants