Skip to content

Comments

chore(internal): remove base error classes (⚠️ breaking)#134

Merged
stainless-bot merged 1 commit intonextfrom
chore-internal-remove-base-error
Sep 22, 2023
Merged

chore(internal): remove base error classes (⚠️ breaking)#134
stainless-bot merged 1 commit intonextfrom
chore-internal-remove-base-error

Conversation

@stainless-bot
Copy link
Collaborator

Migration Guide

If you were instantiating our error classes directly,
you may no longer pass a request kwarg (it is now pulled from the response).

  # before:
- BadRequestError("Test", response=response, request=request)
  # after:
+ BadRequestError("Test", response=response)

## Migration Guide

If you were instantiating our error classes directly,
you may no longer pass a `request` kwarg (it is now pulled from the `response`).

```diff
  # before:
- BadRequestError("Test", response=response, request=request)
  # after:
+ BadRequestError("Test", response=response)
```
@stainless-bot stainless-bot merged commit 8b70672 into next Sep 22, 2023
@stainless-bot stainless-bot deleted the chore-internal-remove-base-error branch September 22, 2023 22:28
@stainless-bot stainless-bot mentioned this pull request Sep 22, 2023
stainless-bot added a commit that referenced this pull request Sep 27, 2023
## Migration Guide
If you were instantiating our error classes directly,
you may no longer pass a `request` kwarg (it is now pulled from the `response`).
```diff
  # before:
- BadRequestError("Test", response=response, request=request)
  # after:
+ BadRequestError("Test", response=response)
```
@stainless-bot stainless-bot mentioned this pull request Sep 27, 2023
@stainless-bot stainless-bot mentioned this pull request Oct 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant