-
-
Notifications
You must be signed in to change notification settings - Fork 96
Upgrade chopper logger deps #403
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
Reviewer's GuideThis PR upgrades the Talker suite to version 5.0.0-dev.10, standardizing dependency versions across packages, documentation, and examples, and adds the Chopper logger release with updated changelogs and publish workflow. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
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.
Hey @Frezyx - I've reviewed your changes and they look great!
Prompt for AI Agents
Please address the comments from this code review:
## Individual Comments
### Comment 1
<location> `utils/version.txt:3` </location>
<code_context>
+# 5.0.0-dev.10
+- - [talker_chopper_logger] Release logger for Chopper http clint package
+
+Thanks to [techouse](https://github.com/techouse)
</code_context>
<issue_to_address>
Typo: 'clint' should be 'client'.
</issue_to_address>
<suggested_fix>
<<<<<<< SEARCH
- [talker_chopper_logger] Release logger for Chopper http clint package
=======
- [talker_chopper_logger] Release logger for Chopper http client package
>>>>>>> REPLACE
</suggested_fix>
### Comment 2
<location> `packages/talker/CHANGELOG.md:2` </location>
<code_context>
+# 5.0.0-dev.10
+- - [talker_chopper_logger] Release logger for Chopper http clint package
+
+Thanks to [techouse](https://github.com/techouse)
</code_context>
<issue_to_address>
Typo: 'clint' should be 'client'.
Update the changelog to fix the typo.
</issue_to_address>
<suggested_fix>
<<<<<<< SEARCH
- - [talker_chopper_logger] Release logger for Chopper http clint package
=======
- - [talker_chopper_logger] Release logger for Chopper http client package
>>>>>>> REPLACE
</suggested_fix>
### Comment 3
<location> `packages/talker_bloc_logger/CHANGELOG.md:2` </location>
<code_context>
+# 5.0.0-dev.10
+- - [talker_chopper_logger] Release logger for Chopper http clint package
+
+Thanks to [techouse](https://github.com/techouse)
</code_context>
<issue_to_address>
Typo: 'clint' should be 'client'.
Update the changelog to correct the typo.
</issue_to_address>
<suggested_fix>
<<<<<<< SEARCH
- - [talker_chopper_logger] Release logger for Chopper http clint package
=======
- - [talker_chopper_logger] Release logger for Chopper http client package
>>>>>>> REPLACE
</suggested_fix>
### Comment 4
<location> `packages/talker_chopper_logger/CHANGELOG.md:2` </location>
<code_context>
+# 5.0.0-dev.10
+- - [talker_chopper_logger] Release logger for Chopper http clint package
+
+Thanks to [techouse](https://github.com/techouse)
</code_context>
<issue_to_address>
Typo: 'clint' should be 'client'.
Please update the changelog to fix the typo.
</issue_to_address>
<suggested_fix>
<<<<<<< SEARCH
- - [talker_chopper_logger] Release logger for Chopper http clint package
=======
- - [talker_chopper_logger] Release logger for Chopper http client package
>>>>>>> REPLACE
</suggested_fix>
### Comment 5
<location> `packages/talker_dio_logger/CHANGELOG.md:2` </location>
<code_context>
+# 5.0.0-dev.10
+- - [talker_chopper_logger] Release logger for Chopper http clint package
+
+Thanks to [techouse](https://github.com/techouse)
</code_context>
<issue_to_address>
Typo: 'clint' should be 'client'.
Update the changelog to correct 'clint' to 'client'.
</issue_to_address>
<suggested_fix>
<<<<<<< SEARCH
- - [talker_chopper_logger] Release logger for Chopper http clint package
=======
- - [talker_chopper_logger] Release logger for Chopper http client package
>>>>>>> REPLACE
</suggested_fix>
### Comment 6
<location> `packages/talker_flutter/CHANGELOG.md:2` </location>
<code_context>
+# 5.0.0-dev.10
+- - [talker_chopper_logger] Release logger for Chopper http clint package
+
+Thanks to [techouse](https://github.com/techouse)
</code_context>
<issue_to_address>
Typo: 'clint' should be 'client'.
Please correct the spelling in the changelog entry.
</issue_to_address>
<suggested_fix>
<<<<<<< SEARCH
- - [talker_chopper_logger] Release logger for Chopper http clint package
=======
- - [talker_chopper_logger] Release logger for Chopper http client package
>>>>>>> REPLACE
</suggested_fix>
### Comment 7
<location> `packages/talker_http_logger/CHANGELOG.md:2` </location>
<code_context>
+# 5.0.0-dev.10
+- - [talker_chopper_logger] Release logger for Chopper http clint package
+
+Thanks to [techouse](https://github.com/techouse)
</code_context>
<issue_to_address>
Typo: 'clint' should be 'client'.
Update the changelog to correct the spelling to 'client'.
</issue_to_address>
<suggested_fix>
<<<<<<< SEARCH
- - [talker_chopper_logger] Release logger for Chopper http clint package
=======
- - [talker_chopper_logger] Release logger for Chopper http client package
>>>>>>> REPLACE
</suggested_fix>
### Comment 8
<location> `packages/talker_logger/CHANGELOG.md:2` </location>
<code_context>
+# 5.0.0-dev.10
+- - [talker_chopper_logger] Release logger for Chopper http clint package
+
+Thanks to [techouse](https://github.com/techouse)
</code_context>
<issue_to_address>
Typo: 'clint' should be 'client'.
Update the changelog to fix the typo.
</issue_to_address>
<suggested_fix>
<<<<<<< SEARCH
- - [talker_chopper_logger] Release logger for Chopper http clint package
=======
- - [talker_chopper_logger] Release logger for Chopper http client package
>>>>>>> REPLACE
</suggested_fix>
### Comment 9
<location> `packages/talker_riverpod_logger/CHANGELOG.md:2` </location>
<code_context>
+# 5.0.0-dev.10
+- - [talker_chopper_logger] Release logger for Chopper http clint package
+
+Thanks to [techouse](https://github.com/techouse)
</code_context>
<issue_to_address>
Typo: 'clint' should be 'client'.
Please update the changelog to correct 'clint' to 'client'.
</issue_to_address>
<suggested_fix>
<<<<<<< SEARCH
- - [talker_chopper_logger] Release logger for Chopper http clint package
=======
- - [talker_chopper_logger] Release logger for Chopper http client package
>>>>>>> REPLACE
</suggested_fix>
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
- [talker_http_logger] Add ``TalkerKey``s registration | ||
Frezyx | ||
5.0.0-dev.10 | ||
- [talker_chopper_logger] Release logger for Chopper http clint package |
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.
issue (typo): Typo: 'clint' should be 'client'.
- [talker_chopper_logger] Release logger for Chopper http clint package | |
- [talker_chopper_logger] Release logger for Chopper http client package |
@@ -1,3 +1,8 @@ | |||
# 5.0.0-dev.10 | |||
- - [talker_chopper_logger] Release logger for Chopper http clint package |
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.
issue (typo): Typo: 'clint' should be 'client'.
Update the changelog to fix the typo.
- - [talker_chopper_logger] Release logger for Chopper http clint package | |
- - [talker_chopper_logger] Release logger for Chopper http client package |
@@ -1,3 +1,8 @@ | |||
# 5.0.0-dev.10 | |||
- - [talker_chopper_logger] Release logger for Chopper http clint package |
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.
issue (typo): Typo: 'clint' should be 'client'.
Update the changelog to correct the typo.
- - [talker_chopper_logger] Release logger for Chopper http clint package | |
- - [talker_chopper_logger] Release logger for Chopper http client package |
@@ -1,3 +1,8 @@ | |||
# 5.0.0-dev.10 | |||
- - [talker_chopper_logger] Release logger for Chopper http clint package |
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.
issue (typo): Typo: 'clint' should be 'client'.
Please update the changelog to fix the typo.
- - [talker_chopper_logger] Release logger for Chopper http clint package | |
- - [talker_chopper_logger] Release logger for Chopper http client package |
@@ -1,3 +1,8 @@ | |||
# 5.0.0-dev.10 | |||
- - [talker_chopper_logger] Release logger for Chopper http clint package |
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.
issue (typo): Typo: 'clint' should be 'client'.
Update the changelog to correct 'clint' to 'client'.
- - [talker_chopper_logger] Release logger for Chopper http clint package | |
- - [talker_chopper_logger] Release logger for Chopper http client package |
@@ -1,3 +1,8 @@ | |||
# 5.0.0-dev.10 | |||
- - [talker_chopper_logger] Release logger for Chopper http clint package |
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.
issue (typo): Typo: 'clint' should be 'client'.
Please correct the spelling in the changelog entry.
- - [talker_chopper_logger] Release logger for Chopper http clint package | |
- - [talker_chopper_logger] Release logger for Chopper http client package |
@@ -1,3 +1,8 @@ | |||
# 5.0.0-dev.10 | |||
- - [talker_chopper_logger] Release logger for Chopper http clint package |
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.
issue (typo): Typo: 'clint' should be 'client'.
Update the changelog to correct the spelling to 'client'.
- - [talker_chopper_logger] Release logger for Chopper http clint package | |
- - [talker_chopper_logger] Release logger for Chopper http client package |
@@ -1,3 +1,8 @@ | |||
# 5.0.0-dev.10 | |||
- - [talker_chopper_logger] Release logger for Chopper http clint package |
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.
issue (typo): Typo: 'clint' should be 'client'.
Update the changelog to fix the typo.
- - [talker_chopper_logger] Release logger for Chopper http clint package | |
- - [talker_chopper_logger] Release logger for Chopper http client package |
@@ -1,3 +1,8 @@ | |||
# 5.0.0-dev.10 | |||
- - [talker_chopper_logger] Release logger for Chopper http clint package |
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.
issue (typo): Typo: 'clint' should be 'client'.
Please update the changelog to correct 'clint' to 'client'.
- - [talker_chopper_logger] Release logger for Chopper http clint package | |
- - [talker_chopper_logger] Release logger for Chopper http client package |
Summary by Sourcery
Release the Chopper HTTP client logger integration and bump all Talker packages to version 5.0.0-dev.10, updating documentation, examples, changelogs, and publication scripts accordingly.
New Features:
Enhancements:
Documentation: