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

fix: Expose spec-assisting tools for authors of external adapters #1134

Merged
merged 1 commit into from Mar 7, 2020

Conversation

olleolleolle
Copy link
Member

@olleolleolle olleolleolle commented Feb 29, 2020

Description

This adds back the require path for "external adapters", which is an assist feature that supports adapter authors.

Details

The one such adapter in existence is faraday-http, which had a failure to build, and that led me here.

I found this out when lostisland/faraday-http#8 failed to build.

When referring to THIS PR as a dependency to that PR: we can get to green - lostisland/faraday-http#8


One thing that users of the feature may need to know is that

gem 'multipart-parser' is needed in their Gemfile.

  - this was inadvertently removed in 0c873c1
  - the feature was introduced in #941
@olleolleolle olleolleolle changed the title External adapters exposed fix: External adapters exposed Feb 29, 2020
@olleolleolle olleolleolle changed the title fix: External adapters exposed (WIP) fix: External adapters exposed Feb 29, 2020
@olleolleolle olleolleolle changed the title (WIP) fix: External adapters exposed fix: External adapters exposed Feb 29, 2020
@olleolleolle olleolleolle changed the title fix: External adapters exposed fix: External adapters exposed again Feb 29, 2020
@olleolleolle olleolleolle changed the title fix: External adapters exposed again fix: Expose spec-assisting tools for authors of external adapters Mar 6, 2020
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.

Doh! Thanks for catching this!

@olleolleolle olleolleolle merged commit 5d24afc into master Mar 7, 2020
@iMacTia iMacTia deleted the add-back-external-adapters-spec-support branch March 9, 2020 12:59
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

2 participants