Skip to content
This repository has been archived by the owner. It is now read-only.

Fix SASS deprecation warnings #58

Merged
merged 1 commit into from Feb 5, 2015
Merged

Fix SASS deprecation warnings #58

merged 1 commit into from Feb 5, 2015

Conversation

@fofr
Copy link
Contributor

@fofr fofr commented Feb 5, 2015

Spotted by @benilovj in the Rails upgrade for release app: alphagov/release#79 (comment)

17:00:35 release.1                 | DEPRECATION WARNING: Extra .css in SCSS file is unnecessary. Rename /usr/lib/rbenv/versions/2.1.4/lib/ruby/gems/2.1.0/gems/govuk_admin_template-1.4.3/app/assets/stylesheets/govuk_admin_template/_buttons.css.scss to /usr/lib/rbenv/versions/2.1.4/lib/ruby/gems/2.1.0/gems/govuk_admin_template-1.4.3/app/assets/stylesheets/govuk_admin_template/_buttons.scss. (called from block in _app_views_layouts_application_html_erb___3941610952330998861_70360721319900 at /var/govuk/release/app/views/layouts/application.html.erb:9)
17:00:35 release.1                 | DEPRECATION WARNING: Extra .css in SCSS file is unnecessary. Rename /usr/lib/rbenv/versions/2.1.4/lib/ruby/gems/2.1.0/gems/govuk_admin_template-1.4.3/app/assets/stylesheets/govuk_admin_template/_nav_list.css.scss to /usr/lib/rbenv/versions/2.1.4/lib/ruby/gems/2.1.0/gems/govuk_admin_template-1.4.3/app/assets/stylesheets/govuk_admin_template/_nav_list.scss. (called from block in _app_views_layouts_application_html_erb___3941610952330998861_70360721319900 at /var/govuk/release/app/views/layouts/application.html.erb:9)
  • Remove the .css part of the sass filenames.
“DEPRECATION WARNING: Extra .css in SCSS file is unnecessary. Rename …”

Remove the .css part of the sass filenames.
bradwright added a commit that referenced this pull request Feb 5, 2015
@bradwright bradwright merged commit d213a5c into master Feb 5, 2015
1 check passed
1 check passed
default "Build #91 succeeded on Jenkins"
Details
@bradwright bradwright deleted the fix-sass-warnings branch Feb 5, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.