-
Notifications
You must be signed in to change notification settings - Fork 3
release: 0.1.4 #111
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.
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
release: 0.1.4 #111
Conversation
6e45d8f
to
bd07c7f
Compare
4aadcd9
to
b90cd33
Compare
28c103c
to
a05398c
Compare
src/finch/_client.py
Outdated
If | ||
Finch’s unified API doesn’t have a data model that cleanly fits your needs, then | ||
Forward allows you to push or pull data models directly against an integration’s | ||
API. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Spacing is off here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actually I'd just remove these lines.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We removed these from now, but you'll need to make the changes you want for documentation like this in your OpenAPI spec before the next update for them to persist. Is that OK?
src/finch/_client.py
Outdated
headers: Optional[object] | NotGiven = NOT_GIVEN, | ||
params: Optional[object] | NotGiven = NOT_GIVEN, | ||
# Use the following arguments if you need to pass additional parameters to the API that aren't available via kwargs. | ||
# The extra values given here take precedence over values defined on the client or passed to this method. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice
51c0f6e
to
ef97220
Compare
|
||
Args: | ||
method: The HTTP method for the forwarded request. Valid values include: `GET` , `POST` | ||
, `PUT` , `DELETE` , and `PATCH`. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nitpick - Can we bring this comma up to be on the previous line?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bump on this @yjp20
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, this was because your OpenAPI spec has spaces after the the backticks; you'll want to fix that up soon.
ce684fd
to
7b68074
Compare
386a7cd
to
03a9546
Compare
ee7ef49
to
0b9561d
Compare
19a3fbc
to
0b9561d
Compare
Automated Release PR
0.1.4 (2023-10-04)
Full Changelog: v0.1.3...v0.1.4
Features
/forward
endpoint and other updates (#116) (ef97220)Chores
This Pull Request has been generated automatically as part of Stainless's release process.
For a better experience, it is recommended to use either rebase-merge or squash-merge when merging this pull request (see details).
More technical details can be found at stainless-api/release-please.