Skip to content

release: 1.71.0#322

Merged
stainless-app[bot] merged 4 commits intomainfrom
release-please--branches--main--changes--next
Feb 28, 2025
Merged

release: 1.71.0#322
stainless-app[bot] merged 4 commits intomainfrom
release-please--branches--main--changes--next

Conversation

@stainless-app
Copy link
Copy Markdown
Contributor

@stainless-app stainless-app Bot commented Feb 28, 2025

Automated Release PR

1.71.0 (2025-02-28)

Full Changelog: v1.70.0...v1.71.0

Features

Chores

Documentation

  • update URLs from stainlessapi.com to stainless.com (#320) (fc1f8bc)

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

@entelligence-ai-pr-reviews
Copy link
Copy Markdown

Walkthrough

The PR includes minor version updates and documentation URL changes in files such as .release-please-manifest.json, CHANGELOG.md, README.md, SECURITY.md, pyproject.toml, and src/julep/_version.py. These changes are primarily related to updating URLs and version numbers.

Entelligence.ai can learn from your feedback. Simply add 👍 / 👎 emojis to teach it your preferences. More shortcuts below

Emoji Descriptions:

  • ⚠️ Potential Issue - May require further investigation.
  • 🔒 Security Vulnerability - Fix to ensure system safety.
  • 💻 Code Improvement - Suggestions to enhance code quality.
  • 🔨 Refactor Suggestion - Recommendations for restructuring code.
  • ℹ️ Others - General comments and information.

Interact with the Bot:

  • Send a message or request using the format:
    @bot + *your message*
Example: @bot Can you suggest improvements for this code?
  • Help the Bot learn by providing feedback on its responses.
    @bot + *feedback*
Example: @bot Do not comment on `save_auth` function !

@entelligence-ai-pr-reviews
Copy link
Copy Markdown

LGTM 👍

@stainless-app stainless-app Bot force-pushed the release-please--branches--main--changes--next branch from 7f01eba to e64cb63 Compare February 28, 2025 03:11
@stainless-app stainless-app Bot force-pushed the release-please--branches--main--changes--next branch from c2f5370 to 58266d0 Compare February 28, 2025 11:13
@stainless-app stainless-app Bot changed the title release: 1.70.1 release: 1.71.0 Feb 28, 2025
@stainless-app stainless-app Bot merged commit a04b0ef into main Feb 28, 2025
@stainless-app stainless-app Bot deleted the release-please--branches--main--changes--next branch February 28, 2025 11:13
@stainless-app
Copy link
Copy Markdown
Contributor Author

stainless-app Bot commented Feb 28, 2025

🤖 Release is at https://github.com/julep-ai/python-sdk/releases/tag/v1.71.0 🌻

Comment on lines 199 to +200
connection_pool: object | NotGiven = NOT_GIVEN,
lang: Literal["en-US"] | NotGiven = NOT_GIVEN,
lang: str | NotGiven = NOT_GIVEN,
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Relaxing lang type from Literal["en-US"] to str allows invalid language codes to be passed without validation, potentially causing runtime errors. Should validate against supported language codes.

📝 Committable Code Suggestion

‼️ Ensure you review the code suggestion before committing it to the branch. Make sure it replaces the highlighted code, contains no missing lines, and has no issues with indentation.

Suggested change
connection_pool: object | NotGiven = NOT_GIVEN,
lang: Literal["en-US"] | NotGiven = NOT_GIVEN,
lang: str | NotGiven = NOT_GIVEN,
lang: Literal["en-US"] | NotGiven = NOT_GIVEN,

mmr_strength: float

mode: Literal["hybrid", "vector", "text"]
mode: str
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

The mode field in RecallOptionsHybridDocSearch should be constrained to Literal["hybrid"] since it's a hybrid-specific class. Current str type allows invalid modes.

📝 Committable Code Suggestion

‼️ Ensure you review the code suggestion before committing it to the branch. Make sure it replaces the highlighted code, contains no missing lines, and has no issues with indentation.

Suggested change
mode: str
mode: Literal["hybrid"]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants