Enable TypeScript's strict mode to fix bugs and preemptively catch ne…

…w ones (#50)

* Fix implicit any type error

* build(tsconfig): Enable strict mode to enforce good practices

* fix: Handle strict mode typescript errors

src/DeviceMotion/DeviceMotion.tsx

@@ -10,10 +10,10 @@ import { SharedRenderProps } from '../types';
 import { isEmptyChildren } from '../utils';
 
 export interface DeviceMotionProps {
-  acceleration: DeviceAcceleration;
-  accelerationIncludingGravity: DeviceAcceleration;
-  rotationRate: DeviceRotationRate;
-  interval: number;
+  acceleration: DeviceAcceleration | null;
+  accelerationIncludingGravity: DeviceAcceleration | null;
+  rotationRate: DeviceRotationRate | null;
+  interval: number | null;
 }
 
 export class DeviceMotion extends React.Component<
@@ -49,11 +49,11 @@ export class DeviceMotion extends React.Component<
   };
 
   componentDidMount() {
-    window.addEventListener('deviceMotion', this.handleDeviceMotion, true);
+    window.addEventListener('devicemotion', this.handleDeviceMotion, true);
   }
 
   componentWillUnmount() {
-    window.removeEventListener('deviceMotion', this.handleDeviceMotion);
+    window.removeEventListener('devicemotion', this.handleDeviceMotion);
   }
 
   render() {

src/Mailto.tsx

@@ -34,8 +34,8 @@ export const Mailto: React.SFC<MailtoProps> = ({
     <a
       href={`mailto:${email}?${qs.stringify({
         subject,
-        cc: cc.join(', '),
-        bcc: bcc.join(', '),
+        cc: cc && cc.join(', '),
+        bcc: bcc && bcc.join(', '),
         body,
       })}`}
       {...props}

src/Scroll/Scroll.tsx

@@ -24,15 +24,15 @@ export class Scroll extends React.Component<
   ScrollConfig & SharedRenderProps<ScrollProps>,
   ScrollProps
 > {
-  static defaultProps = {
+  static defaultProps: Partial<ScrollConfig> = {
     throttle: 100,
   };
 
   state: ScrollProps = { x: 0, y: 0 };
 
   handleWindowScroll = throttle(() => {
     this.setState({ x: window.scrollX, y: window.scrollY });
-  }, this.props.throttle);
+  }, this.props.throttle!);
 
   componentDidMount() {
     this.handleWindowScroll();

src/WindowSize/WindowSize.tsx

@@ -23,15 +23,15 @@ export class WindowSize extends React.Component<
   WindowSizeConfig & SharedRenderProps<WindowSizeProps>,
   WindowSizeProps
 > {
-  static defaultProps = {
-    debounce: 100,
+  static defaultProps: Partial<WindowSizeConfig> = {
+    throttle: 100,
   };
 
   state: WindowSizeProps = { width: 0, height: 0 };
 
   handleWindowSize = throttle(() => {
     this.setState({ width: window.innerWidth, height: window.innerHeight });
-  }, this.props.throttle);
+  }, this.props.throttle!);
 
   componentDidMount() {
     this.handleWindowSize();

src/utils/debounce.ts

@@ -17,7 +17,7 @@
  */
 export const debounce = (func: Function, wait: number, immediate?: boolean) => {
   let timeout: any;
-  return function() {
+  return function(this: any) {
     var context = this,
       args = arguments;
     var later = function() {

src/utils/throttle.ts

@@ -6,7 +6,7 @@
  * @param wait time
  */
 
-export function throttle(func: Function, wait: Number) {
+export function throttle(this: any, func: Function, wait: Number) {
   let timeout: number | null = null;
   let callbackArgs: IArguments | null = null;
   const context = this;

tsconfig.json

@@ -19,6 +19,7 @@
     "removeComments": true,
     "sourceMap": true,
     "pretty": true,
+    "strict": true,
 
     "stripInternal": true,
     "target": "es5",