-
Notifications
You must be signed in to change notification settings - Fork 56
chore: Update text for 'BetaFeaturesNotEnabled' exception. #4581
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
Conversation
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.
Pull Request Overview
Updates the error message for the BetaFeaturesNotEnabled exception to clarify that beta features can be enabled via a Python call to <session>.enable_beta_features(), making it more precise about the enablement method.
- Refactored the exception message construction to use a shared
base_messagevariable - Updated message wording to specify the Python method call syntax with
<session>placeholder
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
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.
Pull Request Overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Context
The exception message was not clear. The message was slightly inaccurate, as it suggested that beta features can only be enabled via a Python call.
Change Summary
The exception message was updated to be more precise about the need of the python call.
Impact
Users will see the updated error message when 'BetaFeaturesNotEnabled' is raised.