new handle session by alexviquez · Pull Request #366 · cuenca-mx/cuenca-python
Navigation Menu
{{ message }}
cuenca-mx / cuenca-python Public
- Notifications You must be signed in to change notification settings
- Fork 14
Draft
Conversation
Copy link Copy Markdown
Contributor
The last implementation had the issue of creating a session context for every request. This has been resolved by creating a Retry adapter. Thanks to @matin for the suggestion
alexviquez added 3 commits
August 23, 2023 12:45
Copy link Copy Markdown
Codecov Report
Merging #366 (bd175cf) into main (2447f60) will not change coverage.
The diff coverage is100.00%.
❗ Current head bd175cf differs from pull request most recent head e543b3d. Consider uploading reports for the commit e543b3d to get more accurate results
@@ Coverage Diff @@ ## main #366 +/- ## ========================================= Coverage 100.00% 100.00% ========================================= Files 47 47 Lines 1082 1087 +5 ========================================= + Hits 1082 1087 +5
| Flag | Coverage Δ | |
|---|---|---|
| unittests | 100.00% <100.00%> (ø) |
Flags with carried forward coverage won't be shown. Click here to find out more.
| Files Changed | Coverage Δ | |
|---|---|---|
| cuenca/http/client.py | 100.00% <100.00%> (ø) |
|
| cuenca/version.py | 100.00% <100.00%> (ø) |
Continue to review full report in Codecov by Sentry.
Legend - Click here to learn more
Δ = absolute <relative> (impact),ø = not affected,? = missing data
Powered by Codecov. Last update 2447f60...e543b3d. Read the comment docs.
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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
1 participant