Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Fix: add requests timeout #70

Merged
merged 2 commits into from
Sep 18, 2023
Merged

Fix: add requests timeout #70

merged 2 commits into from
Sep 18, 2023

Conversation

thekaveman
Copy link
Member

@github-actions
Copy link

Coverage report

Note

Coverage evolution disabled because this PR targets a different branch
than the default branch, for which coverage data is not available.

The coverage rate is 47.7%.
The branch rate is 62%.

100% of new lines are covered.

Diff Coverage details (click to unfold)

eligibility_api/client.py

100% of new lines are covered (97.05% of the complete file).

@thekaveman thekaveman mentioned this pull request Sep 14, 2023
Closed
@thekaveman thekaveman added bug Something isn't working dependencies [auto] Pull requests that update dependencies labels Sep 14, 2023
pre-commit-ci bot and others added 2 commits September 14, 2023 21:41
@pre-commit-ci @thekaveman
chore(pre-commit): autoupdate hooks
536c200 
updates:
- [github.com/compilerla/conventional-pre-commit: v2.1.1 → v2.4.0](compilerla/conventional-pre-commit@v2.1.1...v2.4.0)
- [github.com/psf/black: 23.1.0 → 23.9.1](psf/black@23.1.0...23.9.1)
- [github.com/PyCQA/flake8: 6.0.0 → 6.1.0](PyCQA/flake8@6.0.0...6.1.0)
- [github.com/pycqa/bandit: 1.7.4 → 1.7.5](PyCQA/bandit@1.7.4...1.7.5)
- [github.com/igorshubovych/markdownlint-cli: v0.33.0 → v0.36.0](igorshubovych/markdownlint-cli@v0.33.0...v0.36.0)
@thekaveman
fix(client): timeout parameter with default
6bb9dc2
@thekaveman thekaveman self-assigned this Sep 15, 2023
Base automatically changed from chore/cleanup to main September 15, 2023 15:11
@thekaveman thekaveman marked this pull request as ready for review September 15, 2023 15:12
@thekaveman thekaveman requested a review from a team as a code owner September 15, 2023 15:12
@thekaveman thekaveman merged commit e6135df into main Sep 18, 2023
@thekaveman thekaveman deleted the fix/requests-timeout branch September 18, 2023 16:27
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@angela-tran angela-tran angela-tran approved these changes

Assignees

@thekaveman thekaveman

Labels
bug Something isn't working dependencies [auto] Pull requests that update dependencies
Projects
Digital Services
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@thekaveman @angela-tran