Skip to content

Require minimum faraday version#117

Merged
dblock merged 1 commit intoslack-ruby:masterfrom
dblock:leppert-patch-1
Oct 18, 2016
Merged

Require minimum faraday version#117
dblock merged 1 commit intoslack-ruby:masterfrom
dblock:leppert-patch-1

Conversation

@dblock
Copy link
Copy Markdown
Collaborator

@dblock dblock commented Oct 18, 2016

With Faraday <= 0.8, the following error will be thrown:

slack-ruby-client-0.7.0/lib/slack/web/api/error.rb:4:in `module:Api': superclass must be a Class (Module given) (TypeError)

Replaces #81.

With Faraday `<= 0.8`, the following error will be thrown:

slack-ruby-client-0.7.0/lib/slack/web/api/error.rb:4:in `<module:Api>': superclass must be a Class (Module given) (TypeError)
@dangerpr-bot
Copy link
Copy Markdown

1 Message
📖 We really appreciate pull requests that demonstrate issues, even without a fix. That said, the next step is to try and fix the failing tests!

Generated by 🚫 danger

@dblock dblock merged commit 902172b into slack-ruby:master Oct 18, 2016
@dblock dblock deleted the leppert-patch-1 branch October 18, 2016 17:31
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.

3 participants