Skip to content

Conversation

@Ivanov-Anton
Copy link
Member

@Ivanov-Anton Ivanov-Anton commented Mar 7, 2025

Description

update rack gem from v2.2.12 to v3.1.10

Background

The sprockets gem (v3.7.2), which was previously installed, explicitly required rack to be greater than version 1 and less than version 3. To support the update to rack v3, the decision was made to upgrade sprockets to v4.2.1

Summary of changes

Updated gems

rack gem from v2.2.12 to v3.1.10
rackup gem from v1.0.1 to 2.2.1
sass-rails gem from v5.0.8 to v6.0.0
sassc gem from v3.4.25 to v2.4.0
sprockets gem from v3.7.2 to v4.2.1
rspec_api_documentation gem

  • from repo BigG1947/rspec_api_documentation revision: 269213b

  • to zipmark/rspec_api_documentation v6.1.0

  • because this patch is added

jsonapi-resources gem

  • from v0.9.12 to branch release-0-9

  • because this patch is added (issue, pull request)

Downgraded gems:

thin gem from v1.8.2 to v1.6.2

  • This downgrade was necessary because thin v1.8.2 depends on rack (>= 1, < 3), whereas v1.6.2 is compatible with rack (>= 1.0.0).

Removed gems:

sass-globbing

  • This gem provided an easy way to import multiple Sass or SCSS files in a single import statement. However, since Rails 3.1 and later versions include this functionality by default, the gem is no longer needed.

Additional links

  1. https://github.com/rack/rack/blob/main/UPGRADE-GUIDE.md
  2. https://github.com/rails/sprockets/blob/main/UPGRADING.md
  3. upgrade rack to version 3 #1698

@Ivanov-Anton Ivanov-Anton self-assigned this Mar 7, 2025
@Ivanov-Anton Ivanov-Anton added the dependencies Pull requests that update a dependency file label Mar 7, 2025
@Ivanov-Anton Ivanov-Anton marked this pull request as draft March 7, 2025 21:06
@Ivanov-Anton Ivanov-Anton force-pushed the update-rack branch 2 times, most recently from cb84f2a to d2ff82b Compare March 10, 2025 14:34
@Ivanov-Anton Ivanov-Anton marked this pull request as ready for review March 11, 2025 06:42
@dmitry-sinina
Copy link
Contributor

Could we just remove thin gem?

@dmitry-sinina dmitry-sinina merged commit e97416d into master Mar 11, 2025
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants