Repository navigation
doc, types, test: Use Error objects instead of plain objects with replyWithError() - #2900
Conversation
…en mimicking Node.js errors
|
Thank you @okko! |
|
Same here, I don't know what the use case for that would be, but can't rule it out. I would merge at least #2899 to document usage that better reflects what Node apparently does internally. Then maybe wait with this #2900. On the other hand, reverting this is super easy if someone presents a use case. |
mikicho
left a comment
There was a problem hiding this comment.
Ok, the fix is pretty easy form suer perspective as well, so let's merge this and see if we get any pushback
|
@okko can you please:
|
|
@mikicho Thanks! I fixed formatting and enabled the test. I didn't get the 2nd point of your comment, was there a word missing? |
|
🎉 This PR is included in version 15.0.0-beta.8 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Includes #2899
Fixes #2789