v1.0: enable all rules in tslint-react.json (palantir#28)

* Enable all rules in tslint-react.json

* v1.0.0

* Fix lint failures in jsxAlignmentRule

* Fix lint failures in jsxNoLambdaRule

* Fix lint failures in jsxNoMultilineJsRule

* Fix lint failures in jsxNoStringRefRule

* Fix lint failures in jsxSelfCloseRule

* Enable jsx-no-multiline-js in tslint-react.json

package.json

@@ -1,7 +1,7 @@
 {
   "name": "tslint-react",
-  "version": "0.4.0",
-  "description": "React-specific rules for TSLint",
+  "version": "1.0.0",
+  "description": "Lint rules related to React & JSX for TSLint",
   "main": "tslint-react.json",
   "repository": {
     "type": "git",

src/rules/jsxAlignmentRule.ts

@@ -15,8 +15,8 @@
  * limitations under the License.
  */
 
-import * as ts from "typescript";
 import * as Lint from "tslint/lib/lint";
+import * as ts from "typescript";
 
 export class Rule extends Lint.Rules.AbstractRule {
     public static ATTR_LINE_FAILURE = "JSX attributes must be on a line below the opening tag";
@@ -139,4 +139,3 @@ class JsxAlignmentWalker extends Lint.RuleWalker {
         this.addFailure(this.createFailure(node.getStart(), node.getWidth(), message));
     }
 }
-

src/rules/jsxNoLambdaRule.ts

@@ -15,8 +15,8 @@
  * limitations under the License.
  */
 
-import * as ts from "typescript";
 import * as Lint from "tslint/lib/lint";
+import * as ts from "typescript";
 
 export class Rule extends Lint.Rules.AbstractRule {
     public static FAILURE_STRING = "Lambdas are forbidden in JSX attributes due to their rendering performance impact";

src/rules/jsxNoMultilineJsRule.ts

@@ -15,8 +15,8 @@
  * limitations under the License.
  */
 
-import * as ts from "typescript";
 import * as Lint from "tslint/lib/lint";
+import * as ts from "typescript";
 
 export class Rule extends Lint.Rules.AbstractRule {
     public static FAILURE_STRING = "Multiline JS expressions inside JSX are forbidden";

src/rules/jsxNoStringRefRule.ts

@@ -15,8 +15,9 @@
  * limitations under the License.
  */
 
-import * as ts from "typescript";
 import * as Lint from "tslint/lib/lint";
+import * as ts from "typescript";
+
 import { nodeIsKind } from "../guards";
 
 export class Rule extends Lint.Rules.AbstractRule {

src/rules/jsxSelfCloseRule.ts

@@ -15,8 +15,8 @@
  * limitations under the License.
  */
 
-import * as ts from "typescript";
 import * as Lint from "tslint/lib/lint";
+import * as ts from "typescript";
 
 export class Rule extends Lint.Rules.AbstractRule {
     public static FAILURE_STRING = "JSX elements with no children must be self-closing";

tslint-react.json

@@ -1,3 +1,10 @@
 {
-    "rulesDirectory": "./rules"
+    "rulesDirectory": "./rules",
+    "rules": {
+        "jsx-alignment": true,
+        "jsx-no-lambda": true,
+        "jsx-no-multiline-js": true,
+        "jsx-no-string-ref": true,
+        "jsx-self-close": true
+    }
 }