Skip to content

Conversation

@renefloor
Copy link
Collaborator

Submit a pull request

CLA

  • I have signed the Stream CLA (required).
  • The code changes follow best practices
  • Code changes are tested (add some information if not applicable)

Description of the pull request

This adds classes and helper methods for the http calls using Dio.

@renefloor renefloor force-pushed the feature/dio-http-client branch from 6e85409 to d9500fc Compare August 6, 2025 08:04
@renefloor renefloor changed the title Add classes for Dio Http client feature(llc): Add classes for Dio Http client Aug 6, 2025
@renefloor renefloor changed the title feature(llc): Add classes for Dio Http client feat(llc): Add classes for Dio Http client Aug 6, 2025
@renefloor renefloor force-pushed the feature/dio-http-client branch 2 times, most recently from 9adb0a4 to 5468909 Compare August 6, 2025 08:11
@renefloor renefloor force-pushed the feature/dio-http-client branch from 5468909 to c780e6a Compare August 6, 2025 08:11
@renefloor renefloor force-pushed the feature/dio-http-client branch from e7a5278 to 00a8a4b Compare August 6, 2025 08:27
@codecov
Copy link

codecov bot commented Aug 6, 2025

Codecov Report

❌ Patch coverage is 54.19580% with 131 lines in your changes missing coverage. Please review.
✅ Project coverage is 32.80%. Comparing base (6273bb5) to head (3dc2b2b).
⚠️ Report is 3 commits behind head on main.

Files with missing lines Patch % Lines
...ackages/stream_core/lib/src/logger/stream_log.dart 1.69% 58 Missing ⚠️
...stream_core/lib/src/logger/impl/tagged_logger.dart 0.00% 17 Missing ⚠️
...s/stream_core/lib/src/errors/stream_api_error.dart 38.09% 13 Missing ⚠️
...ages/stream_core/lib/src/logger/stream_logger.dart 7.69% 12 Missing ⚠️
...ackages/stream_core/lib/src/api/token_manager.dart 9.09% 10 Missing ⚠️
packages/stream_core/lib/src/api/http_client.dart 90.14% 7 Missing ⚠️
...m_core/lib/src/api/system_environment_manager.dart 73.33% 4 Missing ⚠️
.../stream_core/lib/src/errors/stream_error_code.dart 57.14% 3 Missing ⚠️
...ream_core/lib/src/ws/client/web_socket_client.dart 0.00% 3 Missing ⚠️
...ore/lib/src/api/interceptors/auth_interceptor.dart 96.29% 1 Missing ⚠️
... and 3 more

❌ Your patch status has failed because the patch coverage (54.19%) is below the target coverage (80.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files
@@             Coverage Diff              @@
##              main       #4       +/-   ##
============================================
- Coverage   100.00%   32.80%   -67.20%     
============================================
  Files            1       30       +29     
  Lines            1      695      +694     
============================================
+ Hits             1      228      +227     
- Misses           0      467      +467     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@renefloor renefloor force-pushed the feature/dio-http-client branch 2 times, most recently from 7a7e787 to 13a4d90 Compare August 6, 2025 09:57
@renefloor renefloor force-pushed the feature/dio-http-client branch from 13a4d90 to 3dc2b2b Compare August 6, 2025 09:58
@renefloor renefloor merged commit eb613bc into main Aug 6, 2025
4 of 7 checks passed
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.

4 participants