Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed bug on naming individual commands with parameter scan #793

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

teofr
Copy link

@teofr teofr commented Jan 5, 2025

When using -P together with --command-name there was a bug when trying to instantiate every individual command name of more than one command.

$ hyperfine  "echo {foo}" "sleep {foo}" -P foo 1 3 -n "echo 1" -n "sleep 1" -n "echo 2" -n "sleep 2"

Would error out with:

Error: '--command-name' has been specified 4 times. It has to appear exactly once, or exactly 3 times (number of benchmarks)

Also, stopped using the hint of the iterator since it could become quite problematic if either gives a wrong hint or None.
This whole bit could use a refactor, but this seemed like a low hanging fruit.

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.

1 participant