feat: -m supports pattern matching #2364
Open
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.
this is a draft of a idea that came to me bumping into projects with multiple mains.
i added printing of multiple mains in last release so user can copy/paste the name - this one uses that same code to let user get to interactively choose which main...but I didn't like it to happen automatically so came up with
-m <pattern>|classname.Basically if
-mis not just a classname we treat it as "ask me".this pr for now just implement
-m ?so you get:could imagine we treat it as a regular expression and you could do:
and
wdyt? too much or just enough to be useful (the code is trivial so feels like worth it to me)
should we do the asking by default or should we require it to be explicilty asked?
note that it uses our standard query mechanism so it will never block - at most delay 30 seconds.