Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Use Error.prototype.cause native concept #2561

Closed
char0n opened this issue Feb 15, 2023 · 0 comments
Closed

Use Error.prototype.cause native concept #2561

char0n opened this issue Feb 15, 2023 · 0 comments
Assignees
@char0n
Labels
ApiDOM enhancement New feature or request

Comments

@char0n
Copy link
Member

char0n commented Feb 15, 2023 •
edited
Loading

The concept is described here: https://nodejs.org/docs/latest-v16.x/api/errors.html#errorcause

Support is aligned here: https://caniuse.com/mdn-javascript_builtins_error_cause

NOTE: As we're building ApiDOM using @babel/present-env against Node.js==12.22.0 we need to be sure we're providing backward compatibility.

Seems like only package using this is apidom-reference, but we need to make sure we'll double check.

How to do the backward compatible extension: https://2ality.com/2021/06/error-cause.html#consequence-for-your-own-code
@char0n char0n self-assigned this Feb 15, 2023
@char0n char0n added enhancement New feature or request ApiDOM labels Feb 15, 2023
char0n added a commit that referenced this issue Feb 15, 2023
@char0n
fix(reference): use JavaScript error causes interface
437e0f5 
Refs #2561
@char0n char0n mentioned this issue Feb 15, 2023
Merged
char0n added a commit that referenced this issue Feb 15, 2023
@char0n
fix(reference): use JavaScript error cause interface (#2564)
f2e1b51 
Refs #2561
@char0n char0n closed this as completed Feb 15, 2023
char0n added a commit that referenced this issue Feb 15, 2023
@char0n
fix(reference): fix extension of native errors
400ffa4 
CausedError was introduced as backward
compatible base error.

Refs #2561
@char0n char0n mentioned this issue Feb 15, 2023
Merged
char0n added a commit that referenced this issue Feb 15, 2023
@char0n
fix(reference): fix extension of native errors (#2565)
afd8905 
CausedError was introduced as backward
compatible base error.

Refs #2561
char0n added a commit to swagger-api/swagger-js that referenced this issue Feb 16, 2023
@char0n
fix(resolver): use native error cause syntax with ApiDOM errors
3664825 
Refs swagger-api/apidom#2561
@char0n char0n mentioned this issue Feb 16, 2023
Merged
char0n added a commit to swagger-api/swagger-js that referenced this issue Feb 16, 2023
@char0n
fix(resolver): use native error cause syntax with ApiDOM errors (#2842)
ae0e7e8 
Refs swagger-api/apidom#2561
@char0n char0n mentioned this issue Jul 10, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@char0n char0n

Labels
ApiDOM enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@char0n