Skip to content

Commit

Permalink
Fix missing name on NonRetriableError (#333)
Browse files Browse the repository at this point in the history
## Summary

Set `NonRetriableError.name`

## Checklist
<!-- Tick these items off as you progress. -->
<!-- If an item isn't applicable, ideally please strikeout the item by
wrapping it in "~~"" and suffix it with "N/A My reason for skipping
this." -->
<!-- e.g. "- [ ] ~~Added tests~~ N/A Only touches docs" -->

- [ ] Added a [docs PR](https://github.com/inngest/website) that
references this PR
- [ ] Added unit/integration tests
- [x] Added changesets if applicable

---------

Co-authored-by: Jack Williams <jack@inngest.com>
  • Loading branch information
amh4r and jpwilliams authored Sep 29, 2023
1 parent a005210 commit 67bda66
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 0 deletions.
5 changes: 5 additions & 0 deletions .changeset/dry-months-sparkle.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
"inngest": patch
---

(Internal) Fix missing name on `NonRetriableError`, ensuring it's correctly (de)serialized
2 changes: 2 additions & 0 deletions packages/inngest/src/components/NonRetriableError.ts
Original file line number Diff line number Diff line change
Expand Up @@ -29,5 +29,7 @@ export class NonRetriableError extends Error {
super(message);
// eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
this.cause = options?.cause;

this.name = "NonRetriableError";
}
}

0 comments on commit 67bda66

Please sign in to comment.