Skip to content

Commit b3c9cdf

Browse files
author
Joachim Seminck
committed
Remove fixer from no-static-typos
1 parent 141e490 commit b3c9cdf

File tree

3 files changed

+3
-39
lines changed

3 files changed

+3
-39
lines changed

docs/rules/no-static-typos.md

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,8 +2,6 @@
22

33
Ensure no casing typos were made declaring static class properties
44

5-
**Fixable:** This rule is automatically fixable using the `--fix` flag on the command line.
6-
75
## Rule Details
86

97
This rule checks whether the declared static class properties related to React components

lib/rules/no-static-typos.js

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -18,19 +18,15 @@ module.exports = {
1818
category: 'Stylistic Issues',
1919
recommended: false
2020
},
21-
fixable: 'code',
2221
schema: []
2322
},
2423

2524
create: Components.detect(function(context, components, utils) {
2625

27-
function reportError(node, classPropertyName) {
26+
function reportError(node) {
2827
context.report({
2928
node: node,
30-
message: 'Typo in static class property declaration',
31-
fix: function(fixer) {
32-
return fixer.replaceText(context.getFirstTokens(node, 2)[1], classPropertyName);
33-
}
29+
message: 'Typo in static class property declaration'
3430
});
3531
}
3632

@@ -47,7 +43,7 @@ module.exports = {
4743

4844
STATIC_CLASS_PROPERTIES.forEach(function(CLASS_PROP) {
4945
if (CLASS_PROP.toLowerCase() === propertyNameLowerCase && CLASS_PROP !== propertyName) {
50-
reportError(node, CLASS_PROP);
46+
reportError(node);
5147
}
5248
});
5349
}

tests/lib/rules/no-static-typos.js

Lines changed: 0 additions & 30 deletions
Original file line numberDiff line numberDiff line change
@@ -103,11 +103,6 @@ ruleTester.run('no-static-typos', rule, {
103103
' static PropTypes = {};',
104104
'}'
105105
].join('\n'),
106-
output: [
107-
'class Component extends React.Component {',
108-
' static propTypes = {};',
109-
'}'
110-
].join('\n'),
111106
parser: 'babel-eslint',
112107
parserOptions: parserOptions,
113108
errors: [{message: ERROR_MESSAGE}]
@@ -117,11 +112,6 @@ ruleTester.run('no-static-typos', rule, {
117112
' static proptypes = {};',
118113
'}'
119114
].join('\n'),
120-
output: [
121-
'class Component extends React.Component {',
122-
' static propTypes = {};',
123-
'}'
124-
].join('\n'),
125115
parser: 'babel-eslint',
126116
parserOptions: parserOptions,
127117
errors: [{message: ERROR_MESSAGE}]
@@ -131,11 +121,6 @@ ruleTester.run('no-static-typos', rule, {
131121
' static ContextTypes = {};',
132122
'}'
133123
].join('\n'),
134-
output: [
135-
'class Component extends React.Component {',
136-
' static contextTypes = {};',
137-
'}'
138-
].join('\n'),
139124
parser: 'babel-eslint',
140125
parserOptions: parserOptions,
141126
errors: [{message: ERROR_MESSAGE}]
@@ -145,11 +130,6 @@ ruleTester.run('no-static-typos', rule, {
145130
' static contexttypes = {};',
146131
'}'
147132
].join('\n'),
148-
output: [
149-
'class Component extends React.Component {',
150-
' static contextTypes = {};',
151-
'}'
152-
].join('\n'),
153133
parser: 'babel-eslint',
154134
parserOptions: parserOptions,
155135
errors: [{message: ERROR_MESSAGE}]
@@ -159,11 +139,6 @@ ruleTester.run('no-static-typos', rule, {
159139
' static DefaultProps = {};',
160140
'}'
161141
].join('\n'),
162-
output: [
163-
'class Component extends React.Component {',
164-
' static defaultProps = {};',
165-
'}'
166-
].join('\n'),
167142
parser: 'babel-eslint',
168143
parserOptions: parserOptions,
169144
errors: [{message: ERROR_MESSAGE}]
@@ -173,11 +148,6 @@ ruleTester.run('no-static-typos', rule, {
173148
' static defaultprops = {};',
174149
'}'
175150
].join('\n'),
176-
output: [
177-
'class Component extends React.Component {',
178-
' static defaultProps = {};',
179-
'}'
180-
].join('\n'),
181151
parser: 'babel-eslint',
182152
parserOptions: parserOptions,
183153
errors: [{message: ERROR_MESSAGE}]

0 commit comments

Comments
 (0)