Rename no-string-ref to jsx-no-string-ref (#11)

README.md

@@ -29,7 +29,7 @@ Sample configuration where `tslint.json` lives adjacent to your `node_modules` f
 - `jsx-no-lambda`
   - Creating new anonymous functions (with either the `function` syntax or ES2015 arrow syntax) inside the `render` call stack works against _pure component rendering_. When doing an equality check between two lambdas, React will always consider them unequal values and force the component to re-render more often than necessary.
   - Rule options: _none_
-- `no-string-ref`
+- `jsx-no-string-ref`
   - Passing strings to the `ref` prop of React elements is considered a legacy feature and will soon be deprecated.
     Instead, [use a callback](https://facebook.github.io/react/docs/more-about-refs.html#the-ref-callback-attribute).
   - Rule options: _none_

src/rules/noStringRefRule.ts → src/rules/jsxNoStringRefRule.ts

@@ -23,22 +23,22 @@ export class Rule extends Lint.Rules.AbstractRule {
     public static FAILURE_STRING = "Pass a callback to ref prop instead of a string literal";
 
     public apply(sourceFile: ts.SourceFile): Lint.RuleFailure[] {
-        const noStringRefWalker = new NoStringRefWalker(sourceFile, this.getOptions());
-        return this.applyWithWalker(noStringRefWalker);
+        const walker = new JsxNoStringRefWalker(sourceFile, this.getOptions());
+        return this.applyWithWalker(walker);
     }
 }
 
-class NoStringRefWalker extends Lint.RuleWalker {
+class JsxNoStringRefWalker extends Lint.RuleWalker {
     protected visitNode(node: ts.Node) {
         if (nodeIsKind<ts.JsxAttribute>(node, ts.SyntaxKind.JsxAttribute)) {
-            const {name, initializer} = node;
+            const { name, initializer } = node;
             const isRefAttribute = name != null && name.text === "ref";
             if (isRefAttribute && initializer != null) {
                 const hasStringInitializer = initializer.kind === ts.SyntaxKind.StringLiteral;
                 const hasStringExpressionInitializer =
                     nodeIsKind<ts.JsxExpression>(initializer, ts.SyntaxKind.JsxExpression)
                     && (initializer.expression.kind === ts.SyntaxKind.StringLiteral
-                    || initializer.expression.kind === ts.SyntaxKind.TemplateExpression);
+                        || initializer.expression.kind === ts.SyntaxKind.TemplateExpression);
 
                 if (hasStringInitializer || hasStringExpressionInitializer) {
                     this.addFailure(this.createFailure(

test/rules/jsx-no-string-ref/tslint.json

@@ -0,0 +1,5 @@
+{
+    "rules": {
+        "jsx-no-string-ref": true
+    }
+}