-
Notifications
You must be signed in to change notification settings - Fork 180
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
asv run "ALL_TAGS" #488
Comments
That would be a really great feature! |
I think this can already be done with |
That doesn't seem to work for me. If I try it errors
If I instead run The problem with this is that it looks like |
Ok maybe |
Unfortunately not, it just runs the latest commit for me if I try the command. |
asv run "`git tag -l | awk '{ print $1 "^!" }'`"is the incantation, but as seen above, not so easy to recall...
|
|
And if I run
instead it does not fail, but it only runs the last commit, not the tags. |
It works fine in bash 4.4.19.
|
Ha, interesting. I am on |
Probably it points towards that the addition of extra ALL_TAGS keywords
could be useful :)
Additionally, "asv run" probably should accept multiple input
arguments, so that one wouldn't need to work hard on getting the
quoting right which apparently depends even on bash version...
|
I do think a ALL_TAGS keyword would be great. But then I have no idea how easy/difficult it is to implement.
Totally agree. Because next I though I just get a list of tags with So thumb up from my side for both, ALL_TAGS and multiple input arguments. |
This seems to work for me: git tag -l | awk '{ print $1 "^!" }' | xargs -L 1 asv run is basically just using xargs as a for loop to loop over every tag in turn. |
If I create a new benchmark I generally want to run the benchmarks for all existing releases (in my case git tags). The
-b
optional already helps to select only the new test but it's either not easy to run on all tags or I missed the option.Nevertheless: I think it would be really nice to have an
"ALL_TAGS"
option equivalent to the already existing"ALL"
,"EXISTING"
and"NEW"
option forrange
inasv run
.The text was updated successfully, but these errors were encountered: