Bugfix: Remove dependency to a namespace JSX of React (#11)

* Add test

* Disable lint in build

Don't want to make too many changes.

* Re-enable allowSynDefault for examples

* Remove dependency to a namespace `JSX` of React

Author: Kohei Asai

.travis.yml

@@ -9,4 +9,4 @@ node_js:
   - "stable"
 
 script:
-  - npm run lint+build+test
+  - npm run verify

examples/active-links.tsx

@@ -1,4 +1,4 @@
-/// <reference path="../typings/main.d.ts" />
+/// <reference path="../typings/index.d.ts" />
 /// <reference path="../react-router.d.ts" />
 
 import React from 'react';
@@ -120,4 +120,4 @@ render((
 			</Route>
 		</Route>
 	</Router>
-), document.getElementById('example'));
+), document.getElementById('example'));

examples/auth.ts

@@ -1,19 +1,25 @@
 export = {
-  login(email?:any, pass?:any, cb?:any) {
-    cb = arguments[arguments.length - 1]
+  login(email?: any, pass?: any, cb?: any) {
+    cb = arguments[arguments.length - 1];
     if (localStorage['token']) {
-      if (cb) cb(true);
-      this.onChange(true)
-      return
+      if (cb) {
+        cb(true);
+      }
+      this.onChange(true);
+      return;
     }
-    pretendRequest(email, pass, (res:any) => {
+    pretendRequest(email, pass, (res: any) => {
       if (res.authenticated) {
-        localStorage['token'] = res.token
-        if (cb) cb(true)
-        this.onChange(true)
+        localStorage['token'] = res.token;
+        if (cb) {
+          cb(true);
+        }
+        this.onChange(true);
       } else {
-        if (cb) cb(false)
-        this.onChange(false)
+        if (cb) {
+          cb(false);
+        }
+        this.onChange(false);
       }
     })
   },
@@ -22,28 +28,30 @@ export = {
     return localStorage['token']
   },
 
-  logout(cb?:any) {
+  logout(cb?: any) {
     delete localStorage['token']
-    if (cb) cb()
-    this.onChange(false)
+    if (cb) {
+      cb();
+    }
+    this.onChange(false);
   },
 
   loggedIn() {
     return !!localStorage['token']
   },
 
-  onChange() {}
-}
+  onChange() { }
+};
 
-function pretendRequest(email:any, pass:any, cb:any) {
+function pretendRequest(email: any, pass: any, cb: any) {
   setTimeout(() => {
     if (email === '[email protected]' && pass === 'password1') {
       cb({
         authenticated: true,
         token: Math.random().toString(36).substring(7)
-      })
+      });
     } else {
-      cb({ authenticated: false })
+      cb({ authenticated: false });
     }
-  }, 0)
-}
+  }, 0);
+}

package.json

@@ -7,10 +7,10 @@
     "source-test": "echo running source tests... &&  echo no source test",
     "test": "echo running server tests... && cd test && ts-node ../node_modules/blue-tape/bin/blue-tape \"**/*.ts\" | tap-spec",
     "watch": "onchange -w \"**/*.ts\" -i -e \"out/**\" -- npm -s run build+test",
-    "publish": "npm -s run lint+build+test && echo please publish to typings/registry",
+    "publish": "npm -s run verify && echo please publish to typings/registry",
     "all-tests": "npm test",
     "build+test": "npm run build && npm run all-tests",
-    "lint+build+test": "npm run lint && npm run build+test",
+    "verify": "npm run build+test",
     "prepublish": "typings install"
   },
   "devDependencies": {

react-router.d.ts

@@ -58,7 +58,7 @@ declare namespace ReactRouter {
 		 * @param {Components<IProps, any>} component
 		 * @param {IProps} props
 		 */
-		createElement?<IProps>(component: ReactComponent<IProps>, props: IProps): JSX.Element;
+		createElement?<IProps>(component: ReactComponent<IProps>, props: IProps): React.ReactElement<IProps>;
 
 		/**
 		 * A function used to convert an object from <Link>s or calls to transitionTo to a URL query string.
@@ -91,7 +91,7 @@ declare namespace ReactRouter {
 		 * This is primarily for integrating with other libraries that need to participate in rendering before the route components are rendered. It defaults to render={(props) => <RouterContext {...props} />}.
 		 * Ensure that you render a <RouterContext> at the end of the line, passing all the props passed to render.
 		 */
-		render?<IProps>(props: IProps): JSX.Element;
+		render?<IProps>(props: IProps): React.ReactElement<IProps>;
 	}
 
 
@@ -785,7 +785,7 @@ declare namespace ReactRouter {
 	/**
 	 * Add router object to props of pure component
 	 */
-	export function withRouter<T>(fun: (props: T & { router: IRouter }) => JSX.Element): (props: T) => JSX.Element;
+	export function withRouter<T>(fun: (props: T & { router: IRouter }) => React.ReactElement<T>): (props: T) => React.ReactElement<T>;
 
 	/**
 	 * Add router object to props of component
@@ -795,7 +795,7 @@ declare namespace ReactRouter {
 	/**
 	 * Apply Router middleware (for using in <Router render={applyRouterMiddleware(...)} ... />)
 	 */
-	export function applyRouterMiddleware<T extends Function>(middleware: T): <IProps>(props: IProps) => JSX.Element;
+	export function applyRouterMiddleware<T extends Function>(middleware: T): <IProps>(props: IProps) => React.ReactElement<IProps>;
 }
 
 export = ReactRouter;

tsconfig.json

@@ -1,5 +1,6 @@
 {
 	"compilerOptions": {
+    "allowSyntheticDefaultImports": true,
 		"target": "es6",
 		"module": "commonjs",
 		"jsx": "react",