Skip to content

Remove unused faraday gems from Gemfile#160

Merged
mateusdeap merged 7 commits intorailsbump:mainfrom
JuanVqz:feature/faraday-ruby4-compat
Apr 9, 2026
Merged

Remove unused faraday gems from Gemfile#160
mateusdeap merged 7 commits intorailsbump:mainfrom
JuanVqz:feature/faraday-ruby4-compat

Conversation

@JuanVqz
Copy link
Copy Markdown
Member

@JuanVqz JuanVqz commented Apr 9, 2026

Summary

  • Remove faraday, faraday_middleware, faraday-multipart, and faraday-retry from the Gemfile
  • None of these were used directly in app code — faraday is only needed as a transitive dependency of octokit (via sawyer)
  • Removing the explicit pins allows faraday to resolve from 1.x to 2.x, dropping the deprecated faraday_middleware and several unused adapter gems

Depends on #159

JuanVqz added 6 commits April 8, 2026 15:51
CGI.parse was removed in Ruby 4.0, causing VCR 6.3.1 to fail.
Unmaintained since 2022 and incompatible with Ruby 4.0 (C extension).
Ruby 4.0 ships with the debug gem as a built-in replacement.
Its dependency rouge uses CGI.parse which is removed in Ruby 4.0.
Rails built-in error pages are sufficient for development.
faraday, faraday_middleware, faraday-multipart, and faraday-retry were
listed explicitly but never used directly in app code. Octokit pulls in
faraday transitively via sawyer. Removing the explicit pins allows
faraday to resolve to 2.x (from 1.x), dropping the deprecated
faraday_middleware and several unused adapter gems.
@JuanVqz JuanVqz marked this pull request as draft April 9, 2026 00:51
@JuanVqz JuanVqz requested review from a team, jeremysmithco and mateusdeap and removed request for jeremysmithco and mateusdeap April 9, 2026 05:00
@JuanVqz JuanVqz marked this pull request as ready for review April 9, 2026 05:03
@JuanVqz JuanVqz self-assigned this Apr 9, 2026
Copy link
Copy Markdown
Contributor

@jeremysmithco jeremysmithco left a comment

Choose a reason for hiding this comment

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

This looks good, too!

@mateusdeap mateusdeap merged commit ce66ad6 into railsbump:main Apr 9, 2026
2 checks passed
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