-
Notifications
You must be signed in to change notification settings - Fork 2
release: 2.20.0 #394
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
base: main
Are you sure you want to change the base?
release: 2.20.0 #394
Conversation
🧪 Testing To try out this version of the SDK, run:
Expires: Fri, 03 Oct 2025 02:10:45 GMT |
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.
Important
Looks good to me! 👍
Reviewed everything up to ebcb576 in 56 seconds. Click for details.
- Reviewed
100
lines of code in6
files - Skipped
0
files when reviewing. - Skipped posting
5
draft comments. View those below. - Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.
1. pyproject.toml:3
- Draft comment:
Version bump to 2.20.0 is applied. Please verify that the dependency ranges remain compatible. - Reason this comment was not posted:
Confidence changes required:0%
<= threshold50%
None
2. src/julep/_version.py:4
- Draft comment:
Version updated correctly to 2.20.0. - Reason this comment was not posted:
Confidence changes required:0%
<= threshold50%
None
3. src/julep/_files.py:49
- Draft comment:
The overload definitions and type-guard functions improve type safety for file inputs. Consider adding tests for edge cases to ensure all supported file types are handled as expected. - Reason this comment was not posted:
Confidence changes required:0%
<= threshold50%
None
4. src/julep/_base_client.py:528
- Draft comment:
The workaround replacing underscores in the host (using host.replace('_', '-')) is clever. Ensure this edge case is well-documented and does not affect hosts that legitimately require underscores. - Reason this comment was not posted:
Confidence changes required:0%
<= threshold50%
None
5. src/julep/_base_client.py:743
- Draft comment:
The exponential backoff with jitter is implemented here. Note that the jitter factor (computed as 1 - 0.25 * random()) only reduces the delay. If a symmetric jitter (allowing slight increases) is desired as the comment suggests 'plus-or-minus half a second', consider adjusting the formula to use a range like random.uniform(0.75, 1.25). - Reason this comment was not posted:
Comment was not on a location in the diff, so it can't be submitted as a review comment.
Workflow ID: wflow_qutBRgSYIP3G21AB
You can customize by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.
ebcb576
to
c830640
Compare
90a412f
to
e6ad37f
Compare
980dff9
to
1c530b2
Compare
841a0ef
to
00c4bf3
Compare
00c4bf3
to
7f2f56e
Compare
7f2f56e
to
b91b39a
Compare
b91b39a
to
823497a
Compare
823497a
to
7c09d7e
Compare
7c09d7e
to
59c535b
Compare
Automated Release PR
2.20.0 (2025-09-03)
Full Changelog: v2.19.2...v2.20.0
Features
Bug Fixes
Chores
This pull request is managed by Stainless's GitHub App.
The semver version number is based on included commit messages. Alternatively, you can manually set the version number in the title of this pull request.
For a better experience, it is recommended to use either rebase-merge or squash-merge when merging this pull request.
🔗 Stainless website
📚 Read the docs
🙋 Reach out for help or questions