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

Switch to sassc #3

Merged
merged 2 commits into from
Jan 17, 2020
Merged

Switch to sassc #3

merged 2 commits into from
Jan 17, 2020

Conversation

n-b
Copy link
Contributor

@n-b n-b commented Nov 29, 2019

As you are probably aware, ruby sass is deprecated. This switches the sass and sass-rails dependencies to their sassc counterparts, and uses the SassC module instead of Sass.

In theory, it just works. At least my limited testing did not reveal issues 😃 .

ℹ️ I also added an empty app/assets/config/manifest.js in the dummy app for rake spec to pass; this is required by Sprockets 4.0.

@shanecav84
Copy link
Collaborator

Thanks! I'll take a look at this this weekend.

@shanecav84 shanecav84 self-requested a review November 29, 2019 19:24
@lubber-de lubber-de added the enhancement New feature or request label Dec 8, 2019
@n-b
Copy link
Contributor Author

n-b commented Jan 17, 2020

Hi @shanecav84! Any chance you might take a look at it?

@shanecav84
Copy link
Collaborator

So sorry! Looking into this now.

@shanecav84 shanecav84 merged commit c4ea3aa into fomantic:master Jan 17, 2020
@shanecav84
Copy link
Collaborator

Released in https://rubygems.org/gems/fomantic-ui-sass/versions/2.8.1.1

@n-b n-b deleted the switch-to-sassc branch January 18, 2020 14:48
n-b added a commit to betagouv/conseillers-entreprises that referenced this pull request Jan 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants