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 runtime dependencies net/http #67

Merged
merged 3 commits into from
Jan 22, 2017

Conversation

gouf
Copy link
Contributor

@gouf gouf commented Jan 22, 2017

実行時の uninitialized constant RakutenWebService::Client::Net (NameError) エラーを修正します

参照: Ruby - Rubyプログラムの楽天APIのNameError(62804)|teratail

Copy link
Member

@satoryu satoryu left a comment

Choose a reason for hiding this comment

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

プルリクエスト送っていただき、ありがとうございます!
非常に助かります 🙇

@@ -17,6 +17,8 @@ Gem::Specification.new do |spec|
spec.require_paths = ["lib"]
spec.required_ruby_version = '>= 2.1.0'

spec.add_runtime_dependency 'net'
Copy link
Member

Choose a reason for hiding this comment

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

これはTwitter関連のgemだと思うのですが、必要なんでしょうか?

https://rubygems.org/gems/net

Copy link
Contributor Author

Choose a reason for hiding this comment

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

すみません、これは余計な追加でした

@@ -1,6 +1,7 @@
module RakutenWebService; end
RWS = RakutenWebService

require 'net/http'
Copy link
Member

Choose a reason for hiding this comment

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

実際にnet/httpを利用するのは、RWS::Client 内なので、lib/rakuten_web_service/client.rb に移した方が関係が明確になって良いかと思います。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

はい、ではそのように変更します!

@satoryu satoryu added the bug label Jan 22, 2017
Copy link
Member

@satoryu satoryu left a comment

Choose a reason for hiding this comment

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

tiqav

@satoryu satoryu merged commit 6dc71de into rakuten-ws:master Jan 22, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants