feat: add deprecation warning system for v30+ api migrations & add curation sets to client #107
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- add suppress_deprecation_warnings field to configdict typeddict - initialize suppress_deprecation_warnings in configuration class - default to false to maintain existing warning behavior
- add warn_deprecation decorator for method deprecation warnings - track shown warnings to prevent duplicate messages - support configurable suppression via suppress_deprecation_warnings - integrate with apicall configuration for warning control
- add warn_deprecation decorator to analytics_rule_v1 and analytics_rules_v1 - add warn_deprecation decorator to override and overrides classes - add warn_deprecation decorator to synonym and synonyms classes - remove manual deprecation warning code and global flags - replace manual logger.warning calls with decorator-based warnings
- add @deprecated decorator to analytics_rule_v1, analytics_v1 classes - add @deprecated decorator to override, overrides, synonyms classes - convert analyticsV1 to private attribute with deprecated property - convert overrides and synonyms to private attributes with deprecated properties - remove manual deprecation warning code from analytics_v1 - enable static type checker warnings for deprecated apis
kishorenc
approved these changes
Nov 14, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Change Summary
warn_deprecationdecorator to logger module for runtime deprecation warningssuppress_deprecation_warningsconfiguration option to control warning visibilitytyping-extensionsdependency for@deprecateddecorator support@deprecateddecorators to deprecated classes (analytics_v1, analytics_rule_v1, override, overrides, synonyms)@warn_deprecationdecorators to deprecated class constructorsanalyticsV1,overrides, andsynonymsto private attributes with deprecated propertieslogger.warncalls withlogger.warningin configuration validationscuration_setsattribute to client classPR Checklist