Skip to content

Autoloading, dependency loading and middleware registry cleanup#1301

Merged
iMacTia merged 2 commits into
mainfrom
legacy-features-cleanup
Aug 9, 2021
Merged

Autoloading, dependency loading and middleware registry cleanup#1301
iMacTia merged 2 commits into
mainfrom
legacy-features-cleanup

Conversation

@iMacTia

@iMacTia iMacTia commented Aug 8, 2021

Copy link
Copy Markdown
Member

Description

  • Remove autoloading
  • Remove adapters from list of dependencies, except for net_http (doesn't require ext libraries and it's good to have a default)
  • Remove dynamic dependency loader
  • Simplify MiddlewareRegistry

Todos

List any remaining work that needs to be done, i.e:

  • Tests
  • Upgrading
  • Documentation?

@iMacTia iMacTia requested a review from olleolleolle August 8, 2021 08:59
@olleolleolle olleolleolle changed the title Autoloading, dependency loading and middlewre registry cleanup Autoloading, dependency loading and middleware registry cleanup Aug 8, 2021

@olleolleolle olleolleolle left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks grand!

I was curious if the middleware_mutex is still in use, now.

Comment thread UPGRADING.md Outdated
Comment thread UPGRADING.md Outdated
Comment thread UPGRADING.md Outdated
Comment thread faraday.gemspec
Comment thread lib/faraday.rb
@iMacTia

iMacTia commented Aug 8, 2021

Copy link
Copy Markdown
Member Author

Thanks for the quick review and the suggestions @olleolleolle 🙌!
Yes middleware_mutex is still used to ensure the middleware stack is thread-safe 👍

@olleolleolle olleolleolle left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This looks ready!

@iMacTia iMacTia merged commit 08b7d4d into main Aug 9, 2021
@iMacTia iMacTia deleted the legacy-features-cleanup branch August 9, 2021 07:25
sgerrand pushed a commit to duffelhq/duffel-api-ruby that referenced this pull request Jan 5, 2022
This class module was refactored and redesigned in lostisland/faraday#1301
timrogers pushed a commit to duffelhq/duffel-api-ruby that referenced this pull request Jan 9, 2022
This class module was refactored and redesigned in lostisland/faraday#1301
timrogers pushed a commit to duffelhq/duffel-api-ruby that referenced this pull request Jan 11, 2022
This class module was refactored and redesigned in lostisland/faraday#1301
timrogers pushed a commit to duffelhq/duffel-api-ruby that referenced this pull request Jan 11, 2022
This class module was refactored and redesigned in lostisland/faraday#1301
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.

2 participants