Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Conditional in BinarySource should check for LDP::Response #85

Merged
merged 1 commit into from
Mar 14, 2018
Merged

Conditional in BinarySource should check for LDP::Response #85

merged 1 commit into from
Mar 14, 2018

Conversation

jcoyne
Copy link
Member

@jcoyne jcoyne commented Jun 15, 2017

Previously it was checking for Faraday::Response

@jcoyne
Conditional in BinarySource should check for LDP::Response
8b56cf4 
Previously it was checking for Faraday::Response
@coveralls
Copy link

coveralls commented Jun 15, 2017 •
edited
Loading

Coverage Status

Coverage remained the same at 82.363% when pulling 8b56cf4 on fix_conditional into e54fe6e on master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage remained the same at 82.363% when pulling 8b56cf4 on fix_conditional into e54fe6e on master.

@jcoyne jcoyne requested a review from cbeer June 15, 2017 17:26
@carrickr carrickr self-requested a review March 14, 2018 21:02
@carrickr carrickr merged commit 021ff6a into master Mar 14, 2018
@tpendragon tpendragon deleted the fix_conditional branch May 8, 2019 18:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cbeer cbeer Awaiting requested review from cbeer

@carrickr carrickr Awaiting requested review from carrickr

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jcoyne @coveralls @carrickr