Skip to content

Commit

Permalink
removes snoflwake tex linting, and lints examples
Browse files Browse the repository at this point in the history
fixes facebookarchive#536

this removes the special tex example linting. that had it's own
copy of eslint linting just itself. after removing eslint i removed
dependencies i didn't see were being used. after that it failed to
build, not reconizing the jsx/import statements. so i added the two
babel presets to handle that. (react, and es2015 presets).

next this starts linting the `examples/` directory. this should
introduce no new warnings/errors. as only the tex example had two
things that didn't comply (trailing commas, which have been fixed).
  • Loading branch information
Eric Coan committed Jul 27, 2016
1 parent 9ffe701 commit 92991b8
Show file tree
Hide file tree
Showing 4 changed files with 6 additions and 186 deletions.
1 change: 0 additions & 1 deletion .eslintignore
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
lib/
dist/
docs/
examples/
node_modules/
website/
172 changes: 0 additions & 172 deletions examples/tex/.eslintrc

This file was deleted.

9 changes: 3 additions & 6 deletions examples/tex/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -6,12 +6,9 @@
},
"dependencies": {
"babel": "5.8.23",
"babel-eslint": "^4.1.3",
"babel-loader": "5.3.2",
"classnames": "^2.1.3",
"eslint": "^1.0.0",
"eslint-loader": "^1.0.0",
"eslint-plugin-react": "^3.2.0",
"babel-loader": "^6.2.4",
"babel-preset-es2015": "^6.9.0",
"babel-preset-react": "^6.11.1",
"express": "^4.13.1",
"immutable": "^3.7.4",
"katex": "^0.5.1",
Expand Down
10 changes: 3 additions & 7 deletions examples/tex/server.js
Original file line number Diff line number Diff line change
Expand Up @@ -22,19 +22,15 @@ const APP_PORT = 3000;
// Serve the TeX Editor app
var compiler = webpack({
entry: path.resolve(__dirname, 'js', 'app.js'),
eslint: {
configFile: '.eslintrc',
},
module: {
loaders: [
{
test: /\.js$/,
exclude: /node_modules/,
loader: 'babel',
},
{
test: /\.js$/,
loader: 'eslint',
query: {
presets: ['es2015', 'react'],
},
},
],
},
Expand Down

0 comments on commit 92991b8

Please sign in to comment.