Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: package.json

Issue 29808587: Issue 6717 - Part 1: run eslint through npm (Closed) Base URL: https://hg.adblockplus.org/adblockpluschrome/
Patch Set: Use eslint4 and eyeo eslint 2.0.0. Rename task to "lint" Created July 13, 2018, 2:14 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « README.md ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: package.json
===================================================================
new file mode 100644
--- /dev/null
+++ b/package.json
@@ -0,0 +1,14 @@
+{
+ "name": "adblockpluschrome",
+ "repository": "https://hg.adblockplus.org/adblockpluschrome",
+ "license": "GPL-3.0",
+ "dependencies": {},
+ "devDependencies": {
+ "eslint": "^4.19.0",
Sebastian Noack 2018/07/13 15:04:08 We want to update to ESLint 5, don't we? Moreover,
hub 2018/07/13 15:05:47 I have moved on to ^5.1.0. When the peerDependency
Sebastian Noack 2018/07/13 15:08:26 Let's update eslint-config-eyeo [1] first, so that
hub 2018/07/13 16:12:43 Acknowledged.
Sebastian Noack 2018/08/14 19:16:55 What is about this comment?
hub 2018/08/14 19:35:16 it is needed. Without it, I get: ```` $ npm insta
+ "eslint-config-eyeo": "2.0.0"
+ },
+ "scripts": {
+ "lint": "eslint *.js lib/ qunit/ ext/",
+ "posttest": "npm run lint"
+ }
+}
« no previous file with comments | « README.md ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld