Skip to content

Remove explicit sprockets dependency for Rails 7 #2530

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

Merged
merged 1 commit into from
Nov 17, 2021

Conversation

pirj
Copy link
Member

@pirj pirj commented Nov 16, 2021

https://github.com/rspec/rspec-rails/runs/4222116982?check_suite_focus=true

    [!] There was an error parsing `Gemfile`:
    [!] There was an error parsing `Gemfile-rails-dependencies`: You cannot specify the same gem twice with different version requirements.
    You specified: sprockets-rails (>= 2.0.0) and sprockets-rails (>= 0). Bundler cannot continue.

     #  from /home/runner/work/rspec-rails/rspec-rails/Gemfile-rails-dependencies:10
     #  -------------------------------------------
     #    gem 'sprockets', :git => 'https://github.com/rails/sprockets.git', :branch => 'master'
     >    gem 'sprockets-rails', :git => 'https://github.com/rails/sprockets-rails.git', :branch => 'master'

    [!] There was an error parsing `Gemfile`:
    [!] There was an error parsing `Gemfile-rails-dependencies`: You cannot specify the same gem twice with different version requirements.
    You specified: sprockets-rails (>= 2.0.0) and sprockets-rails (>= 0). Bundler cannot continue.

     #  from /home/runner/work/rspec-rails/rspec-rails/Gemfile-rails-dependencies:10
     #  -------------------------------------------
     #    gem 'sprockets', :git => 'https://github.com/rails/sprockets.git', :branch => 'master'
     >    gem 'sprockets-rails', :git => 'https://github.com/rails/sprockets-rails.git', :branch => 'master'
@pirj pirj self-assigned this Nov 16, 2021
@pirj pirj added the rails7 label Nov 16, 2021
Comment on lines -6 to -8
gem "activerecord-deprecated_finders", :git => "https://github.com/rails/activerecord-deprecated_finders.git"
gem "web-console", :git => "https://github.com/rails/web-console", :group => :development
gem 'coffee-rails', :git => "https://github.com/rails/coffee-rails.git"
Copy link
Member Author

Choose a reason for hiding this comment

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

Do we need this?

@pirj pirj marked this pull request as ready for review November 16, 2021 18:44
@pirj
Copy link
Member Author

pirj commented Nov 16, 2021

Rails 7 build is not entirely green, but at least it runs. #2521 is supposed to fix some of the failures.

@pirj pirj requested review from benoittgt and JonRowe November 16, 2021 18:45
@JonRowe
Copy link
Member

JonRowe commented Nov 16, 2021

@pirj I don't have time to debug Rails 7 / master stuff right now, merge this if you're happy :)

@pirj pirj merged commit d3e7b85 into main Nov 17, 2021
@pirj pirj deleted the fix-edge-rails-sprockets-duplicate-dependency branch November 17, 2021 12:22
@pirj pirj added this to the 6.0 milestone Jan 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants