dragosmg commented on pull request #11534:
URL: https://github.com/apache/arrow/pull/11534#issuecomment-962983868
I'm not sure what the best name for the helper(s) would be (as I think we
might need more than one). Initially I went with `err()` for the
`expect_snapshot()` + `(expect_error())` pattern, but I think a similar
approach is useful for `expect_warning()` too. Naming the helper `warn()` would
conflict with {rlang} so I went with `err_helper()` and `warn_helper()` as
names.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
