Skip to content

Conversation

@wolfy1339
Copy link
Member

Resolves #ISSUE_NUMBER


Before the change?

  • The options didn't inherit from the base Error class

After the change?

  • RequestErrorOptions now extends ErrorOptions

Pull request checklist

  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been reviewed and added / updated if needed (for bug fixes / features)

Does this introduce a breaking change?

Please see our docs on breaking changes to help!

  • Yes
  • No

@wolfy1339 wolfy1339 added the Type: Feature New feature or request label Nov 13, 2025
@octokit
Copy link

octokit bot commented Nov 13, 2025

👋 Hi! Thank you for this contribution! Just to let you know, our GitHub SDK team does a round of issue and PR reviews twice a week, every Monday and Friday! We have a process in place for prioritizing and responding to your input. Because you are a part of this community please feel free to comment, add to, or pick up any issues/PRs that are labeled with Status: Up for grabs. You & others like you are the reason all of this works! So thank you & happy coding! 🚀

options: RequestErrorOptions,
) {
super(message);
super(message, { cause: options.cause });
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this be simplified to

Suggested change
super(message, { cause: options.cause });
super(message, options});

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No, because

Image

Copy link
Contributor

@Uzlopak Uzlopak left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@wolfy1339 wolfy1339 changed the title feat: inherit options from base Error class feat: inherit options from base Error class to add support for the cause property Nov 13, 2025
wolfy1339 added a commit that referenced this pull request Nov 13, 2025
…`cause` property (#535/#536)

* feat: inherit options from base Error class
* chore(deps): update dependency esbuild to ^0.27.0

---------

Co-authored-by: wolfy1339 <[email protected]>
Co-authored-by: wolfy1339 <[email protected]>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@wolfy1339
Copy link
Member Author

Merged via #535

@wolfy1339 wolfy1339 closed this Nov 13, 2025
@wolfy1339 wolfy1339 deleted the inherit-erroroptions branch November 13, 2025 18:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Type: Feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants