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.

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

Add HTTP.rb adapter #1

Merged
merged 14 commits into from Mar 11, 2019
Merged

Add HTTP.rb adapter #1

merged 14 commits into from Mar 11, 2019

Conversation

iMacTia
Copy link
Member

@iMacTia iMacTia commented Mar 8, 2019

Description

This adds an adapter for HTTP - The Gem! a.k.a http.rb
Based on the work from @PandaWhisperer in PandaWhisperer/faraday#1

Copy link
Member

@technoweenie technoweenie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some small issues, but this looks solid.

lib/faraday/http/adapter.rb Outdated Show resolved Hide resolved
lib/faraday/http/adapter.rb Outdated Show resolved Hide resolved
technoweenie and others added 3 commits March 8, 2019 17:56
Co-Authored-By: iMacTia <iMacTia@users.noreply.github.com>
Register the adapter under symbol :http
@@ -37,8 +37,14 @@ Gem::Specification.new do |spec|
spec.executables = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
spec.require_paths = ['lib']

# spec.add_dependency 'faraday', '~> 1.0'
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
# spec.add_dependency 'faraday', '~> 1.0'

@iMacTia
Copy link
Member Author

iMacTia commented Mar 9, 2019

Just waiting for lostisland/faraday#941 to be merged before merging this in, will need to change the Gemfile as well

@olleolleolle
Copy link
Member

@iMacTia That branch is now merged, this PR is ready for its next step: Gemfile fix.

@iMacTia
Copy link
Member Author

iMacTia commented Mar 10, 2019

Thanks! Will try to have a look today otherwise tomorrow will be done 😁

@iMacTia iMacTia merged commit e7c3403 into master Mar 11, 2019
@iMacTia iMacTia deleted the add-adapter branch March 11, 2019 09:17
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.

None yet

3 participants