Comments (2)
Hey @katgironpe, thanks for using scss-lint
!
This is a great idea. We thought about it a while back, but at the time weren't sure about its practicality as it's possible it too would result in a lot of noise. Now that we've moved completely onto SCSS, we've noticed some clear patterns where deeply nested selectors (whether they're child or descendant combinators) not only result in poor performance, but are also a smell. The magic number for us seems to be 3, as in a depth > 3 is too deep.
Regarding the comment about typos/spaces, I would agree that it is not a valuable use of time to correct an existing codebase to such a style for the sake of doing so. At @causes we value consistency, especially when we can create an automated tool to enforce such a consistency for us, and think that this does indeed bring value.
However, if the more stylistic lints are bugging you, not to worry. We'll soon be releasing a configuration system that allows you to enable/disable particular lints.
Thanks for your comments and suggestions. We'll try to take a stab at this sometime this week.
from scss-lint.
The stylistic lints are not bugging me. I do not disagree with anything the gem does so far.
from scss-lint.
Related Issues (20)
- Ruby version 3.2.0 HOT 3
- Extend separate_groups options in PropertySortOrder
- gem "scss-lint" installs scss-lint-0.38.0 HOT 1
- Single control comment does not work in at least one edge case HOT 1
- scss-lint output newscss HOT 1
- PseudoElement does not work with ::placeholder property HOT 2
- Feature Request: Reused Classname/Html name throws errors
- HexValidation - Allow 4-digit hex values HOT 1
- Exclude files instead of disabling linter when generating config HOT 1
- How to exclude node_modules? HOT 1
- Document multiple exclude globs
- scss-lint not compatible with IntelliJ IDEA Educational Edition 2019.3 x64 HOT 2
- Support --fix like eslint --fix to autofix whitespace related formatting lint issues HOT 1
- sass-lint is unmaintained HOT 2
- Request: add scroll-margin property to default whitelist HOT 2
- Make it possible to define required line breaks before blocks and modifiers in BEM notation
- Problems with scrollbar-color HOT 1
- Unknown property warning for 'gap' and 'row-gap' HOT 1
- Properties `text-underline-offset` and `text-decoration-thickness` false-positives
- ::marker should be valid
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from scss-lint.