Comments (6)
Sounds good. One question: For e.g. number-leading-zero
should the negative instead use "unexpected": "Unexpected leading zero for fractional value less than 1"??
from stylelint.
One question: For e.g. number-leading-zero should the negative instead use "unexpected": "Unexpected leading zero for fractional value less than 1"??
Yeap, I had that same thought just after I clicked "Comment"! :)
That works for me if we make the other "always"|"never" rules consistent too e.g.
declaration-colon/bang-space
: "Expected single space before "!"" and "Unexpected space before "!"declaration-block-trailing-semicolon
: "Expected a trailing semicolon" and "Unexpected trailing semicolon"number-leading-zero
: "Expected a leading zero for fractional value less than 1" and "Unexpected leading zero for fractional value less than 1"
When it comes to rules with values other than "always" and "never" I think we should tend towards "expected" over "unexpected" e.g.
url-quotes: "single"|"double"|none
We might need to create multiple rejected messages for each value if we use "unexpected" e.g.
For url-quotes: "single"
We'd need two rejected messages:
- "Unexpected double quotes for URL"
- "Unexpected lack of quotes for URL"
Where as, the "expected" approach would require just one rejected message for each value:
e.g. for url-quotes: "single"
We'd only need:
- "Expected single quotes for URL"
Sound good?
from stylelint.
Yes. I think "unexpected" (aka "rejected") is for situations where what's there is not ok but there is not a specific expected alternative. So url-quotes
never "rejects" if it's on, because it's always expecting something specific. We're in agreement.
from stylelint.
We're in agreement.
Good stuff :)
from stylelint.
👍
from stylelint.
I can do this.
from stylelint.
Related Issues (20)
- Update the documentation of `isAutoprefixable` util
- Address potentially breaking changes in #7769 HOT 5
- Autofix refactors HOT 40
- Make fixer data testable
- Ranges of fixes HOT 14
- `function-no-unknown` should check at rules too HOT 16
- Add stylelint-config-adblock as a native stylelint rule HOT 1
- Media query interpolation false positive HOT 9
- Fix `stylelint.utils.checkAgainstRule()` for Promise-based rules HOT 1
- Release 16.7.0 HOT 11
- Constructing replacement text for fixes without bleed or side effects HOT 2
- New repository for testing ecosystem HOT 11
- Valid disables reported when `--report-needless-disables` and `--quiet` are set HOT 8
- `no-descending-specificity` is very confusing when it comes to nested rules HOT 6
- Replace `result.warn` by `emitDeprecationWarning` in `lintPostcssResult.mjs` HOT 3
- Fix `no-descending-specificity` false positives for nested rules without declarations HOT 4
- Fix `function-url-quotes` false positives for SCSS variable and `#` character HOT 2
- New rule to enforce `@scope` usage HOT 5
- Write more documentation for post processor HOT 5
- Add `name` configuration property under `overrides` HOT 5
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 stylelint.