Merge pull request #334 from vzhou842/flow-strict

Enable Flow Strict

.flowconfig

@@ -12,3 +12,8 @@ module.name_mapper.extension='scss' -> '<PROJECT_ROOT>/flow/css-module-stub.js'
 server.max_workers=4
 
 [strict]
+nonstrict-import
+unclear-type
+untyped-type-import
+unsafe-getters-setters
+sketchy-null

flow/css-module-stub.js

@@ -1,4 +1,4 @@
-// @flow
+// @flow strict
 type CSSModule = {
   [key: string]: string,
 };

src/cms/index.js

@@ -1,4 +1,4 @@
-// @flow
+// @flow strict
 import CMS from 'netlify-cms-app';
 import PagePreview from './preview-templates/page-preview';
 import PostPreview from './preview-templates/post-preview';

src/cms/preview-templates/page-preview.js

@@ -1,4 +1,4 @@
-// @flow
+// @flow strict
 import React from 'react';
 import type { Entry, WidgetFor } from '../../types';
 

src/cms/preview-templates/post-preview.js

@@ -1,4 +1,4 @@
-// @flow
+// @flow strict
 import React from 'react';
 import type { Entry, WidgetFor } from '../../types';
 

src/components/Feed/Feed.js

@@ -1,4 +1,4 @@
-// @flow
+// @flow strict
 import React from 'react';
 import moment from 'moment';
 import { Link } from 'gatsby';

src/components/Feed/Feed.test.js

@@ -1,4 +1,4 @@
-// @flow
+// @flow strict
 import React from 'react';
 import renderer from 'react-test-renderer';
 import Feed from './Feed';

src/components/Feed/index.js

@@ -1,2 +1,2 @@
-// @flow
+// @flow strict
 export { default } from './Feed';

src/components/Icon/Icon.js

@@ -1,4 +1,4 @@
-// @flow
+// @flow strict
 import React from 'react';
 import styles from './Icon.module.scss';
 

src/components/Icon/Icon.test.js

@@ -1,4 +1,4 @@
-// @flow
+// @flow strict
 import React from 'react';
 import renderer from 'react-test-renderer';
 import Icon from './Icon';

src/components/Icon/index.js

@@ -1,2 +1,2 @@
-// @flow
+// @flow strict
 export { default } from './Icon';

src/components/Layout/Layout.js

@@ -1,4 +1,4 @@
-// @flow
+// @flow strict
 import React from 'react';
 import Helmet from 'react-helmet';
 import type { Node as ReactNode } from 'react';

src/components/Layout/Layout.test.js

@@ -1,4 +1,4 @@
-// @flow
+// @flow strict
 import React from 'react';
 import renderer from 'react-test-renderer';
 import Layout from './Layout';

src/components/Layout/index.js

@@ -1,2 +1,2 @@
-// @flow
+// @flow strict
 export { default } from './Layout';

src/components/Page/Page.test.js

@@ -1,4 +1,4 @@
-// @flow
+// @flow strict
 import React from 'react';
 import renderer from 'react-test-renderer';
 import Page from './Page';

src/components/Page/index.js

@@ -1,2 +1,2 @@
-// @flow
+// @flow strict
 export { default } from './Page';

src/components/Pagination/Pagination.js

@@ -1,4 +1,4 @@
-// @flow
+// @flow strict
 import React from 'react';
 import classNames from 'classnames/bind';
 import { Link } from 'gatsby';

src/components/Pagination/Pagination.test.js

@@ -1,4 +1,4 @@
-// @flow
+// @flow strict
 import React from 'react';
 import renderer from 'react-test-renderer';
 import Pagination from './Pagination';

src/components/Pagination/index.js

@@ -1,2 +1,2 @@
-// @flow
+// @flow strict
 export { default } from './Pagination';

src/components/Post/Author/Author.js

@@ -1,4 +1,4 @@
-// @flow
+// @flow strict
 import React from 'react';
 import { getContactHref } from '../../../utils';
 import styles from './Author.module.scss';

src/components/Post/Author/Author.test.js

@@ -1,4 +1,4 @@
-// @flow
+// @flow strict
 import React from 'react';
 import renderer from 'react-test-renderer';
 import { useStaticQuery, StaticQuery } from 'gatsby';

src/components/Post/Author/index.js

@@ -1,2 +1,2 @@
-// @flow
+// @flow strict
 export { default } from './Author';

src/components/Post/Comments/Comments.js

@@ -1,4 +1,4 @@
-// @flow
+// @flow strict
 import React from 'react';
 import ReactDisqusComments from 'react-disqus-comments';
 import { useSiteMetadata } from '../../../hooks';

src/components/Post/Comments/Comments.test.js

@@ -1,4 +1,4 @@
-// @flow
+// @flow strict
 import React from 'react';
 import renderer from 'react-test-renderer';
 import { useStaticQuery, StaticQuery } from 'gatsby';

src/components/Post/Comments/index.js

@@ -1,2 +1,2 @@
-// @flow
+// @flow strict
 export { default } from './Comments';

src/components/Post/Content/Content.js

@@ -1,4 +1,4 @@
-// @flow
+// @flow strict
 import React from 'react';
 import styles from './Content.module.scss';
 

src/components/Post/Content/Content.test.js

@@ -1,4 +1,4 @@
-// @flow
+// @flow strict
 import React from 'react';
 import renderer from 'react-test-renderer';
 import Content from './Content';

src/components/Post/Content/index.js

@@ -1,2 +1,2 @@
-// @flow
+// @flow strict
 export { default } from './Content';

src/components/Post/Meta/Meta.js

@@ -1,4 +1,4 @@
-// @flow
+// @flow strict
 import React from 'react';
 import moment from 'moment';
 import styles from './Meta.module.scss';

src/components/Post/Meta/Meta.test.js

@@ -1,4 +1,4 @@
-// @flow
+// @flow strict
 import React from 'react';
 import renderer from 'react-test-renderer';
 import Meta from './Meta';

src/components/Post/Meta/index.js

@@ -1,2 +1,2 @@
-// @flow
+// @flow strict
 export { default } from './Meta';

src/components/Post/Post.js

@@ -1,4 +1,4 @@
-// @flow
+// @flow strict
 import React from 'react';
 import { Link } from 'gatsby';
 import Author from './Author';

src/components/Post/Post.test.js

@@ -1,4 +1,4 @@
-// @flow
+// @flow strict
 import React from 'react';
 import renderer from 'react-test-renderer';
 import { useStaticQuery, StaticQuery } from 'gatsby';

src/components/Post/Tags/Tags.js

@@ -1,4 +1,4 @@
-// @flow
+// @flow strict
 import React from 'react';
 import { Link } from 'gatsby';
 import styles from './Tags.module.scss';

src/components/Post/Tags/Tags.test.js

@@ -1,4 +1,4 @@
-// @flow
+// @flow strict
 import React from 'react';
 import renderer from 'react-test-renderer';
 import Tags from './Tags';

src/components/Post/Tags/index.js

@@ -1,2 +1,2 @@
-// @flow
+// @flow strict
 export { default } from './Tags';

src/components/Post/index.js

@@ -1,2 +1,2 @@
-// @flow
+// @flow strict
 export { default } from './Post';

src/components/Sidebar/Author/Author.js

@@ -1,4 +1,4 @@
-// @flow
+// @flow strict
 import React from 'react';
 import { withPrefix, Link } from 'gatsby';
 import styles from './Author.module.scss';
@@ -24,7 +24,7 @@ const Author = ({ author, isIndex }: Props) => (
       />
     </Link>
 
-    { isIndex ? (
+    { isIndex === true ? (
       <h1 className={styles['author__title']}>
         <Link className={styles['author__title-link']} to="/">{author.name}</Link>
       </h1>

src/components/Sidebar/Author/Author.test.js

@@ -1,4 +1,4 @@
-// @flow
+// @flow strict
 import React from 'react';
 import renderer from 'react-test-renderer';
 import Author from './Author';

src/components/Sidebar/Author/index.js

@@ -1,2 +1,2 @@
-// @flow
+// @flow strict
 export { default } from './Author';

src/components/Sidebar/Contacts/Contacts.js

@@ -1,4 +1,4 @@
-// @flow
+// @flow strict
 import React from 'react';
 import { getContactHref, getIcon } from '../../../utils';
 import Icon from '../../Icon';

src/components/Sidebar/Contacts/Contacts.test.js

@@ -1,5 +1,5 @@
 
-// @flow
+// @flow strict
 import React from 'react';
 import renderer from 'react-test-renderer';
 import Contacts from './Contacts';

src/components/Sidebar/Contacts/index.js

@@ -1,2 +1,2 @@
-// @flow
+// @flow strict
 export { default } from './Contacts';

src/components/Sidebar/Copyright/Copyright.js

@@ -1,4 +1,4 @@
-// @flow
+// @flow strict
 import React from 'react';
 import styles from './Copyright.module.scss';
 
@@ -12,4 +12,4 @@ const Copyright = ({ copyright }: Props) => (
   </div>
 );
 
-export default Copyright;
+export default Copyright;

src/components/Sidebar/Copyright/Copyright.test.js

@@ -1,4 +1,4 @@
-// @flow
+// @flow strict
 import React from 'react';
 import renderer from 'react-test-renderer';
 import Copyright from './Copyright';

src/components/Sidebar/Copyright/index.js

@@ -1,2 +1,2 @@
-// @flow
+// @flow strict
 export { default } from './Copyright';

src/components/Sidebar/Menu/Menu.js

@@ -1,4 +1,4 @@
-// @flow
+// @flow strict
 import React from 'react';
 import { Link } from 'gatsby';
 import styles from './Menu.module.scss';

src/components/Sidebar/Menu/Menu.test.js

@@ -1,4 +1,4 @@
-// @flow
+// @flow strict
 import React from 'react';
 import renderer from 'react-test-renderer';
 import Menu from './Menu';

src/components/Sidebar/Menu/index.js

@@ -1,2 +1,2 @@
-// @flow
+// @flow strict
 export { default } from './Menu';

src/components/Sidebar/Sidebar.js

@@ -1,4 +1,4 @@
-// @flow
+// @flow strict
 import React from 'react';
 import Author from './Author';
 import Contacts from './Contacts';

src/components/Sidebar/Sidebar.test.js

@@ -1,4 +1,4 @@
-// @flow
+// @flow strict
 import React from 'react';
 import renderer from 'react-test-renderer';
 import { useStaticQuery, StaticQuery } from 'gatsby';

src/components/Sidebar/index.js

@@ -1,2 +1,2 @@
-// @flow
+// @flow strict
 export { default } from './Sidebar';

src/constants/icons.js

@@ -1,4 +1,4 @@
-// @flow
+// @flow strict
 const ICONS = {
   TWITTER: {
     path: 'M25.312 6.375c-0.688 1-1.547 1.891-2.531 2.609 0.016 0.219 0.016 0.438 0.016 0.656 0 6.672-5.078 14.359-14.359 14.359-2.859 0-5.516-0.828-7.75-2.266 0.406 0.047 0.797 0.063 1.219 0.063 2.359 0 4.531-0.797 6.266-2.156-2.219-0.047-4.078-1.5-4.719-3.5 0.313 0.047 0.625 0.078 0.953 0.078 0.453 0 0.906-0.063 1.328-0.172-2.312-0.469-4.047-2.5-4.047-4.953v-0.063c0.672 0.375 1.453 0.609 2.281 0.641-1.359-0.906-2.25-2.453-2.25-4.203 0-0.938 0.25-1.797 0.688-2.547 2.484 3.062 6.219 5.063 10.406 5.281-0.078-0.375-0.125-0.766-0.125-1.156 0-2.781 2.25-5.047 5.047-5.047 1.453 0 2.766 0.609 3.687 1.594 1.141-0.219 2.234-0.641 3.203-1.219-0.375 1.172-1.172 2.156-2.219 2.781 1.016-0.109 2-0.391 2.906-0.781z',

src/constants/index.js

@@ -1,3 +1,3 @@
-// @flow
+// @flow strict
 export { default as ICONS } from './icons';
 export { default as PAGINATION } from './pagination';

src/constants/pagination.js

@@ -1,4 +1,4 @@
-// @flow
+// @flow strict
 const PAGINATION = {
   PREV_PAGE: '← PREV',
   NEXT_PAGE: '→ NEXT'

src/hooks/index.js

@@ -1,4 +1,4 @@
-// @flow
+// @flow strict
 export { default as useSiteMetadata } from './use-site-metadata';
 export { default as useCategoriesList } from './use-categories-list';
 export { default as useTagsList } from './use-tags-list';

src/hooks/use-categories-list.js

@@ -1,4 +1,4 @@
-// @flow
+// @flow strict
 import { useStaticQuery, graphql } from 'gatsby';
 
 const useCategoriesList = () => {

src/hooks/use-site-metadata.js

@@ -1,4 +1,4 @@
-// @flow
+// @flow strict
 import { useStaticQuery, graphql } from 'gatsby';
 
 const useSiteMetadata = () => {

src/hooks/use-tags-list.js

@@ -1,4 +1,4 @@
-// @flow
+// @flow strict
 import { useStaticQuery, graphql } from 'gatsby';
 
 const useTagsList = () => {

src/templates/categories-list-template.js

@@ -1,4 +1,4 @@
-// @flow
+// @flow strict
 import React from 'react';
 import { Link } from 'gatsby';
 import kebabCase from 'lodash/kebabCase';
@@ -29,4 +29,4 @@ const CategoriesListTemplate = () => {
   );
 };
 
-export default CategoriesListTemplate;
+export default CategoriesListTemplate;

src/templates/categories-list-template.test.js

@@ -1,4 +1,4 @@
-// @flow
+// @flow strict
 import React from 'react';
 import renderer from 'react-test-renderer';
 import { useStaticQuery, StaticQuery } from 'gatsby';

src/templates/category-template.js

@@ -1,4 +1,4 @@
-// @flow
+// @flow strict
 import React from 'react';
 import { graphql } from 'gatsby';
 import Layout from '../components/Layout';

src/templates/category-template.test.js

@@ -1,4 +1,4 @@
-// @flow
+// @flow strict
 import React from 'react';
 import renderer from 'react-test-renderer';
 import { useStaticQuery, StaticQuery } from 'gatsby';

src/templates/index-template.js

@@ -1,4 +1,4 @@
-// @flow
+// @flow strict
 import React from 'react';
 import { graphql } from 'gatsby';
 import Layout from '../components/Layout';

src/templates/index-template.test.js

@@ -1,4 +1,4 @@
-// @flow
+// @flow strict
 import React from 'react';
 import renderer from 'react-test-renderer';
 import { StaticQuery, useStaticQuery } from 'gatsby';

src/templates/not-found-template.js

@@ -1,4 +1,4 @@
-// @flow
+// @flow strict
 import React from 'react';
 import Sidebar from '../components/Sidebar';
 import Layout from '../components/Layout';

src/templates/not-found-template.test.js

@@ -1,4 +1,4 @@
-// @flow
+// @flow strict
 import React from 'react';
 import renderer from 'react-test-renderer';
 import { useStaticQuery, StaticQuery } from 'gatsby';

src/templates/page-template.js

@@ -1,4 +1,4 @@
-// @flow
+// @flow strict
 import React from 'react';
 import { graphql } from 'gatsby';
 import Layout from '../components/Layout';

src/templates/page-template.test.js

@@ -1,4 +1,4 @@
-// @flow
+// @flow strict
 import React from 'react';
 import renderer from 'react-test-renderer';
 import { useStaticQuery, StaticQuery } from 'gatsby';
@@ -27,4 +27,4 @@ describe('PageTemplate', () => {
     const tree = renderer.create(<PageTemplate {...props} />).toJSON();
     expect(tree).toMatchSnapshot();
   });
-});
+});

src/templates/post-template.js

@@ -1,4 +1,4 @@
-// @flow
+// @flow strict
 import React from 'react';
 import { graphql } from 'gatsby';
 import Layout from '../components/Layout';

src/templates/post-template.test.js

@@ -1,4 +1,4 @@
-// @flow
+// @flow strict
 import React from 'react';
 import renderer from 'react-test-renderer';
 import { useStaticQuery, StaticQuery } from 'gatsby';
@@ -27,4 +27,4 @@ describe('PostTemplate', () => {
     const tree = renderer.create(<PostTemplate {...props} />).toJSON();
     expect(tree).toMatchSnapshot();
   });
-});
+});

src/templates/tag-template.js

@@ -1,4 +1,4 @@
-// @flow
+// @flow strict
 import React from 'react';
 import { graphql } from 'gatsby';
 import Layout from '../components/Layout';

src/templates/tag-template.test.js

@@ -1,4 +1,4 @@
-// @flow
+// @flow strict
 import React from 'react';
 import renderer from 'react-test-renderer';
 import { useStaticQuery, StaticQuery } from 'gatsby';
@@ -29,4 +29,4 @@ describe('TagTemplate', () => {
     const tree = renderer.create(<TagTemplate {...props} />).toJSON();
     expect(tree).toMatchSnapshot();
   });
-});
+});

src/templates/tags-list-template.js

@@ -1,4 +1,4 @@
-// @flow
+// @flow strict
 import React from 'react';
 import { Link } from 'gatsby';
 import kebabCase from 'lodash/kebabCase';
@@ -29,4 +29,4 @@ const TagsListTemplate = () => {
   );
 };
 
-export default TagsListTemplate;
+export default TagsListTemplate;

src/templates/tags-list-template.test.js

@@ -1,4 +1,4 @@
-// @flow
+// @flow strict
 import React from 'react';
 import renderer from 'react-test-renderer';
 import { useStaticQuery, StaticQuery } from 'gatsby';

src/types/index.js

@@ -1,7 +1,8 @@
-// @flow
+// @flow strict
 import type { Node as ReactNode } from 'react';
 
 export type RenderCallback = {
+  // $FlowFixMe
   render: (data: any) => ReactNode;
 }
 

src/utils/get-contact-href.js

@@ -1,4 +1,4 @@
-// @flow
+// @flow strict
 const getContactHref = (name: string, contact: string) => {
   let href;
 

src/utils/get-contact-href.test.js

@@ -1,4 +1,4 @@
-// @flow
+// @flow strict
 import getContactHref from './get-contact-href';
 
 test('getContactHref', () => {

src/utils/get-icon.js

@@ -1,4 +1,4 @@
-// @flow
+// @flow strict
 import { ICONS } from '../constants';
 
 const getIcon = (name: string) => {

src/utils/get-icon.test.js

@@ -1,4 +1,4 @@
-// @flow
+// @flow strict
 import getIcon from './get-icon';
 import { ICONS } from '../constants';
 

src/utils/index.js

@@ -1,3 +1,3 @@
-// @flow
+// @flow strict
 export { default as getIcon } from './get-icon';
 export { default as getContactHref } from './get-contact-href';