Skip to content
This repository has been archived by the owner on Mar 19, 2024. It is now read-only.

Merge -match with the shell command to run #22

Open
mvdan opened this issue Jul 3, 2017 · 0 comments
Open

Merge -match with the shell command to run #22

mvdan opened this issue Jul 3, 2017 · 0 comments

Comments

@mvdan
Copy link
Owner

mvdan commented Jul 3, 2017

So this:

goreduce -match 'crash string' . "go build"

Would become:

goreduce . "go build |& grep 'crash string'"

And we would want a zero exit code. (the |& is just to match stderr too)

Advantages:

  • Simpler tool (one less flag)
  • The logic is no longer tied/restricted to a regex

Disadvantages:

  • Basic shell commands become more complex
  • Most usages can be expressed in terms of a regex (I think?)

Leaning towards doing this, just thinking outloud.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant