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

update to sassc-rails #287

Open
wants to merge 5 commits into
base: develop
Choose a base branch
from

Conversation

chnikki
Copy link

@chnikki chnikki commented Dec 19, 2019

Keep getting the sass eol error when running bundle install. The sass gem reached end of life March 26, 2019. This change is switching to use sassc-rails, which uses the new C implementation of Sass as requested in issue #278. I also added rails 6.0 to the appraisal test.

Chantele Parker added 2 commits December 19, 2019 13:16
Keep getting the sass eol error when running bundle install. The sass gem reached end of life March 26, 2019. This change is switching to use sassc-rails, which uses the new C implementation of Sass as requested in issue foundation#278. I also added rails 6.0 to the appraisal test.
Getting this error in Travis CI:
Name: bundler
  Version: 2.1.0
  File: bundler-2.1.0.gem
bundler's executable "bundle" conflicts with /home/travis/.rvm/rubies/ruby-2.3.8/bin/bundle
Overwrite the executable? [yN]

specify the bundler version in travis.yml so it matches what's in the Gemfile.lock.
Copy link

@javierjulio javierjulio left a comment

Choose a reason for hiding this comment

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

@cparker-ajla thanks a lot for doing this! ❤️ I saw that the builds were failing but they shouldn't be. I've suggested some small changes to see if we can get them passing. Thanks!

@@ -203,8 +212,8 @@ DEPENDENCIES
railties
rake
rspec (~> 3.2)
sass

Choose a reason for hiding this comment

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

With sass being removed here, I still see sass listed further up in the gem specs section of the lock file. I would think it would be removed from there too. Was there no other changes with this lock file? The gemspec file was updated but not sure why each of the rails lock files have sass specified still.

.travis.yml Outdated

before_install:
- gem update --system --no-doc
- gem install bundler
- gem install -v 1.17.3 bundler --no-rdoc --no-ri
- bundle _1.17.3_ install

Choose a reason for hiding this comment

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

We should remove this as Travis will run this command automatically with the specific version of bundler we have installed.

Suggested change
- bundle _1.17.3_ install

.travis.yml Outdated

before_install:
- gem update --system --no-doc
- gem install bundler
- gem install -v 1.17.3 bundler --no-rdoc --no-ri

Choose a reason for hiding this comment

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

The build is failing but I'm not sure if its getting to this point. Could you please update the before_install to the following and push to see if it helps at all?

before_install:
  - gem update --system 
  - gem install bundler:1.17.3

Choose a reason for hiding this comment

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

Yikes, sorry about that @cparker-ajla. Could you try this instead:

  - gem install bundler -v "1.17.3"

Copy link
Author

Choose a reason for hiding this comment

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

Thank you for all your help getting past this error @javierjulio!

Chantele Parker added 3 commits January 3, 2020 15:09
try to fix failing build as suggested by @javierjulio
remove extra space
fix to bundler version
@javierjulio
Copy link

@cparker-ajla thanks! Now we are getting a different error. I think the problem may be related to the gemspec path and gemfiles based on what I look up for that error message. Here is a sample error from one of the builds:

You are trying to install in deployment mode after changing
your Gemfile. Run `bundle install` elsewhere and add the
updated gemfiles/rails_5.0.gemfile.lock to version control.

The gemspecs for path gems changed

Copy link
Author

@chnikki chnikki left a comment

Choose a reason for hiding this comment

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

I'll try to get get these specs updated correctly. I don't know why sass is still showing in them.

@javierjulio
Copy link

No problem! Thanks for tackling this. Let me know if you have an update and/or need help. I'm not familiar with appraisals so I'm not sure why the builds are failing. The the lock files in /gemfiles don't look right though so I'm thinking something went wrong with updating them perhaps?

@p8 p8 mentioned this pull request Feb 13, 2020
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