Make --include
and --exclude
use exact matching.
#1138
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Currently they do substring matching. But because some benchmarks have a
name that is a substring of another it's impossible to match some
benchmarks by themselves. E.g.:
webrender
? You'll getwebrender-wrench
too.syn
? You'll getdeeply-nested-async
too.deeply-nested
? You'll getdeeply-nested-async
anddeeply-nested-closures
too.This commit makes the matching exact. As a heavy user of these tools I
think this is a good trade-off. It makes some things possible that are
currently impossible (e.g. including or excluding just
webrender
), atthe cost of more typing on occasion, e.g.
--include ctfe-stress-4
instead of
--include ctfe
.(Note: if you specify something that doesn't match any benchmark's name
the command will give an error.)