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

Add a configuration file for scss-lint #3854

Closed
wants to merge 1 commit into from
Closed

Add a configuration file for scss-lint #3854

wants to merge 1 commit into from

Conversation

ghost
Copy link

@ghost ghost commented Feb 11, 2015

Add a configuration file for scss linting.
It is for scss-lint (https://github.com/causes/scss-lint).

The rules needs to be discussed by core team IMHO.

Linting rules are taken from aldryn guidelines (http://aldryn-boilerplate-standard.readthedocs.org/en/latest/css/guidelines.html).
I am a frontend developer and I set the other rules to sane values (in my opinion).

@yakky
Copy link
Member

yakky commented Feb 11, 2015

@FinalAngel we can discuss such linting rules and then @carloratm will handle the conversion

@yakky yakky added this to the 3.1 milestone Feb 11, 2015
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.01%) to 90.89% when pulling 23d99da on carloratm:scss-lint into 560767f on divio:develop.

@coveralls
Copy link

coveralls commented Feb 11, 2015

Coverage Status

Coverage decreased (-0.1%) to 90.795% when pulling 23d99da on carloratm:scss-lint into 560767f on divio:develop.

@FinalAngel
Copy link
Member

We have already some linting rules and agreed on following: https://github.com/aldryn/aldryn-boilerplate-bootstrap3 at some point (post 3.1) @yakky

@FinalAngel FinalAngel assigned yakky and unassigned FinalAngel Mar 4, 2015
@yakky yakky modified the milestones: 3.1, 3.2 Mar 17, 2015
@yakky
Copy link
Member

yakky commented Jul 10, 2015

Superseded by #4246

@yakky yakky closed this Jul 10, 2015
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.

None yet

4 participants