-
Notifications
You must be signed in to change notification settings - Fork 8
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
README: Show which rules have options, and which are fixable #289
Conversation
Codecov Report
@@ Coverage Diff @@
## master #289 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 102 102
Lines 607 607
=========================================
Hits 607 607 |
@@ -81,11 +81,14 @@ The following global settings can be used under the `no-jquery` property to conf | |||
|
|||
## 📖 Rules | |||
|
|||
Where rules are included in the configs `recommended`, `slim`, `all` or `deprecated-X.X` it is indicated below. Where rules are included with options this is indicated with a `†`. | |||
Rules with options are indicated with ⚙️. Rules which are fixable are indicated with 🔧. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Quote this with `s like we do on the next line for †.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The dagger is escaped because it becomes part of the config label: slim†
and the escaping allows us to show the multi-character label as one graphical item. The emoji don't need this, it would just add a pointless background: 🔧
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It also messes up the emoji rendering of the gear: ⚙️
vs ⚙️
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
They all look terrible to me (possibly an artefact of me using GItHub dark mode?).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
aacd3cf
to
2e1efe1
Compare
No description provided.