Skip to content

Allow multiple benchmarks per folder in ruby-bench#445

Merged
rafaelfranca merged 1 commit into
mainfrom
rmf-multiple-benches-per-folder
Nov 27, 2025
Merged

Allow multiple benchmarks per folder in ruby-bench#445
rafaelfranca merged 1 commit into
mainfrom
rmf-multiple-benches-per-folder

Conversation

@rafaelfranca

Copy link
Copy Markdown
Collaborator

Today, benchmarks are organized, one per folder. For libraries like addressable, we have 9 folders with repeated Gemfile and Gemfile.lock.

This PR, adds the possibility of adding benchmarks for the same library into the same folder, each with a different name.

@rafaelfranca rafaelfranca requested a review from a team November 21, 2025 21:19
Comment thread README.md Outdated
@k0kubun

k0kubun commented Nov 21, 2025

Copy link
Copy Markdown
Member

I wonder if we need any change for https://github.com/rubybench/rubybench and/or https://github.com/rubybench/rubybench.github.io to support this. cc: @eightbitraptor

@rafaelfranca

Copy link
Copy Markdown
Collaborator Author

I wonder if we need any change for https://github.com/rubybench/rubybench and/or https://github.com/rubybench/rubybench.github.io to support this. cc: @eightbitraptor

It should not. The data generate is in the same format with the same keys. The way to execute is also the same.

@rafaelfranca rafaelfranca force-pushed the rmf-multiple-benches-per-folder branch from 00b07bd to 2d1d48b Compare November 21, 2025 22:40

@k0kubun k0kubun 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.

👍

@eightbitraptor eightbitraptor 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.

This looks good to me. Thanks 👍

@rafaelfranca rafaelfranca force-pushed the rmf-multiple-benches-per-folder branch from 2d1d48b to cd1365d Compare November 27, 2025 18:40
@rafaelfranca rafaelfranca merged commit a8cd2d6 into main Nov 27, 2025
13 checks passed
@rafaelfranca rafaelfranca deleted the rmf-multiple-benches-per-folder branch November 27, 2025 18:54
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.

3 participants