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

Use !default for SASS variables #2777

Merged
merged 1 commit into from
Oct 3, 2017

Conversation

peschee
Copy link
Contributor

@peschee peschee commented Oct 2, 2017

In order to be able to override the variables in SASS, each variable declaration should have a !default flag (http://sass-lang.com/documentation/file.SASS_REFERENCE.html#Variable_Defaults___default).

In order to be able to override the variables in SASS, each variable declaration should have a `!default` flag (http://sass-lang.com/documentation/file.SASS_REFERENCE.html#Variable_Defaults___default).
@codecov
Copy link

codecov bot commented Oct 2, 2017

Codecov Report

Merging #2777 into development will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##           development    #2777   +/-   ##
============================================
  Coverage        70.49%   70.49%           
============================================
  Files              137      137           
  Lines             4088     4088           
  Branches           710      710           
============================================
  Hits              2882     2882           
  Misses             908      908           
  Partials           298      298

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b9ee706...a164f5d. Read the comment docs.

@valorkin
Copy link
Member

valorkin commented Oct 3, 2017

@peschee thanks for PR!

@EvilAlexei please review this changes, if it's looks good -> approve

@valorkin valorkin added this to the v2.0.0 milestone Oct 3, 2017
Copy link
Contributor

@EvilAlexei EvilAlexei left a comment

Choose a reason for hiding this comment

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

Look's good, thank you!

@valorkin valorkin merged commit cf2365a into valor-software:development Oct 3, 2017
@peschee
Copy link
Contributor Author

peschee commented Oct 3, 2017

❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants