-
Notifications
You must be signed in to change notification settings - Fork 7
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
Swap requests with httpx #19
Conversation
@branchedelac @chadmcinnis @jalexsc If one of you could take a look at this PR, it would be much appreciated |
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.
The changes look good. Do you know why some of the tests are failing?
@branchedelac I think it has to do with how the test runs are configured. I was looking at it yesterday, but I was going to create a new branch to try out a fix. I will say I've been using it with my 1.8.0 migration tool release candidates, and it's been fine. Though, I don't think I've done anything that requires using the folio_get_all method, yet.
|
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
Closes #16 and removes deprecated loans and requests-specific methods.