-
Notifications
You must be signed in to change notification settings - Fork 826
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
Sdk minor release 2024 #839
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #839 +/- ##
==========================================
- Coverage 89.82% 89.81% -0.01%
==========================================
Files 19 19
Lines 31987 32159 +172
==========================================
+ Hits 28732 28885 +153
- Misses 3255 3274 +19 ☔ View full report in Codecov by Sentry. |
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.
Left a question about style
sdk_headers = get_sdk_headers(service_name=self.DEFAULT_SERVICE_NAME, | ||
service_version='V1', | ||
operation_id='list_synonyms') | ||
sdk_headers = get_sdk_headers( | ||
service_name=self.DEFAULT_SERVICE_NAME, | ||
service_version='V1', | ||
operation_id='list_synonyms', | ||
) |
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.
It looks like the vast majority of these changes are indent changes. Is this an intentional stylistic change or did we forget to run a style linter?
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.
This is due to changes to updates to the sdk generator
BREAKING CHANGE: Name changes for multiple classes
c82d672
to
f982a82
Compare
🎉 This PR is included in version 8.0.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
No description provided.