Skip to content

Conversation

@jsonbailey
Copy link
Contributor

@jsonbailey jsonbailey commented Sep 29, 2025

Note

Low Risk
Only adds a deprecation warning and documentation note for Config.copy_with_new_sdk_key, with no behavioral changes aside from emitting DeprecationWarning at call sites.

Overview
Marks Config.copy_with_new_sdk_key as deprecated by adding an explicit docstring notice and emitting a DeprecationWarning (via warnings.warn(..., stacklevel=2)) when the method is called.

Written by Cursor Bugbot for commit 09cec6d. This will update automatically on new commits. Configure here.

@jsonbailey jsonbailey requested a review from a team as a code owner September 29, 2025 21:55
@jsonbailey
Copy link
Contributor Author

Cursorbot called out this issue in a separate PR where it was not related to the work being done.

@jsonbailey jsonbailey changed the title fix: Add missing config values to copy operation fix: Mark copy_with_new_sdk_key method as deprecated Feb 4, 2026
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

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.

2 participants