Skip to content

Commit

Permalink
Update create-rule script to allow choosing fixable "whitespace" (#934
Browse files Browse the repository at this point in the history
)
  • Loading branch information
fisker committed Dec 12, 2020
1 parent 17bf27c commit 2ecc766
Show file tree
Hide file tree
Showing 3 changed files with 21 additions and 13 deletions.
26 changes: 17 additions & 9 deletions scripts/create-rule.js
Original file line number Diff line number Diff line change
Expand Up @@ -110,10 +110,10 @@ function updateReadmeRules(data) {
const RULE_START = '## Rules\n\n';
const RULE_END = '\n\n## Deprecated Rules';
let ruleContent = `- [${data.id}](docs/rules/${data.id}.md) - ${data.description}`;
if (data.isFixable === true) {
ruleContent += ' *(fixable)*';
} else if (data.isFixable === 'partly') {
if (data.isFixable && data.isFixable.partly) {
ruleContent += ' *(partly fixable)*';
} else if (data.isFixable) {
ruleContent += ' *(fixable)*';
}

const file = path.join(ROOT, 'readme.md');
Expand Down Expand Up @@ -178,16 +178,24 @@ function updateReadme(data) {
message: 'Is it fixable?',
choices: [
{
name: true,
message: 'Yes'
message: 'Code',
value: {type: 'code'}
},
{
message: 'Code (Partly)',
value: {type: 'code', partly: true}
},
{
message: 'Whitespace',
value: {type: 'whitespace'}
},
{
name: 'partly',
message: 'Partly'
message: 'Whitespace (Partly)',
value: {type: 'whitespace', partly: true}
},
{
name: false,
message: 'No'
message: 'No',
value: false
}
]
},
Expand Down
6 changes: 3 additions & 3 deletions scripts/template/documentation.md.jst
Original file line number Diff line number Diff line change
@@ -1,10 +1,10 @@
# <%= description %>

<!-- More detailed description. Remove this comment. -->
<% if (isFixable === true) { %>
This rule is fixable.
<% } else if (isFixable === 'partly') { %>
<% if (isFixable && isFixable.partly) { %>
This rule is partly fixable.
<% } else if (isFixable) { %>
This rule is fixable.
<% } %>
## Fail

Expand Down
2 changes: 1 addition & 1 deletion scripts/template/rule.js.jst
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ module.exports = {
docs: {
url: getDocumentationUrl(__filename)
},<% if (isFixable) { %>
fixable: 'code',<% } %>
fixable: '<%= isFixable.type %>',<% } %>
schema,
messages
}
Expand Down

0 comments on commit 2ecc766

Please sign in to comment.