Skip to content

Commit

Permalink
Enforce flow declarations on all js files
Browse files Browse the repository at this point in the history
  • Loading branch information
gregtatum committed Jul 13, 2018
1 parent 83114bc commit 9f83623
Show file tree
Hide file tree
Showing 7 changed files with 7 additions and 3 deletions.
4 changes: 1 addition & 3 deletions .eslintrc.js
Original file line number Diff line number Diff line change
Expand Up @@ -48,9 +48,7 @@ module.exports = {
{ ignorePureComponents: true },
],
'react/jsx-no-bind': 'error',
// We still have too many files without the file annotation. Let's enable it
// once we flow-type everything.
// 'flowtype/require-valid-file-annotation': [ 'error', 'always', { annotationStyle: 'line' } ],
'flowtype/require-valid-file-annotation': [ 'error', 'always', { annotationStyle: 'line' } ],
// no-dupe-keys crashes with recent eslint. See
// https://github.com/gajus/eslint-plugin-flowtype/pull/266 and
// https://github.com/gajus/eslint-plugin-flowtype/pull/302
Expand Down
1 change: 1 addition & 0 deletions bin/check-warn-yarn-changed.js
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
// @noflow
// MIT © Sindre Sorhus - sindresorhus.com
// via https://gist.github.com/sindresorhus/7996717

Expand Down
1 change: 1 addition & 0 deletions bin/post-checkout.js
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
// @noflow
const [origHead, head, flag] = process.env.GIT_PARAMS.split(' ');

// Flag is 1 if we moved between branches. Flag is 0 if we merely checked out a file from another branch.
Expand Down
1 change: 1 addition & 0 deletions bin/post-merge.js
Original file line number Diff line number Diff line change
@@ -1 +1,2 @@
// @noflow
require('./check-warn-yarn-changed.js')('ORIG_HEAD', 'HEAD');
1 change: 1 addition & 0 deletions bin/post-rewrite.js
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
// @noflow
// This is either 'rebase' or 'amend'.
if (process.env.GIT_PARAMS !== 'rebase') {
process.exit();
Expand Down
1 change: 1 addition & 0 deletions server.js
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
// @noflow
const webpack = require('webpack');
const WebpackDevServer = require('webpack-dev-server');
const config = require('./webpack.config');
Expand Down
1 change: 1 addition & 0 deletions webpack.config.js
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
// @noflow
const path = require('path');
const webpack = require('webpack');
const HtmlWebpackPlugin = require('html-webpack-plugin');
Expand Down

0 comments on commit 9f83623

Please sign in to comment.