fatih / errwrap

Go tool to wrap and fix errors with the new %w verb directive
https://arslan.io
Other
371 stars 16 forks source link

Fix bug to allow detecting multiple wrap directives #9

Closed tariq1890 closed 4 years ago

tariq1890 commented 4 years ago

Thanks for this project @fatih, I was writing test cases in an effort to understanding the project and I was unable to replicate the scenario for multiple %w directives. This bug fix allows for it.

Happy to hear your thoughts on this

tariq1890 commented 4 years ago

@fatih Would you be able to review this?

fatih commented 4 years ago

Looks good. Thanks for your contribution 👍