Upgrade Loop's use of eslint (eslint version 1.5.1, eslint-plugin-react version 3.5.1)

RESOLVED FIXED in Firefox 44

Status

P3
normal
Rank:
30
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: standard8, Assigned: standard8)

Tracking

unspecified
mozilla44
Points:
---

Firefox Tracking Flags

(firefox44 fixed)

Details

(Whiteboard: [tech-debt])

Attachments

(1 attachment)

For each library:

- Library versions are maintained in browser/components/loop/standalone/package.json
- Review the release notes from the current library version to the new version
-- Look out for any significant changes/additions/breaking changes
-- Eslint release notes listed on blog: http://eslint.org/blog/
-- Eslint-plugin-react list at: https://github.com/yannickcr/eslint-plugin-react/releases
- Update the library versions in the package.json file
- Run `make install` in the standalone directory to install the new library version
- Run eslint:
-- `cd browser/components/loop`
-- `standalone/node_modules/.bin/eslint --ext .js --ext .jsm --ext .jsx .

- Ensure no errors occur, fixing them if they do occur.

Occasionally a library will have an error that's an actual regression, and if so it then needs filing with the library, and typically we hold off the upgrade until it is fixed.

- Submit the patch for review, reference any of the major changes spotted.
- When it lands, send notification of any major changes to loop-frontend-dev. For new features, we're generally only enabling them if we know we really want them to aid review etc.
Rank: 25

Updated

3 years ago
Rank: 25 → 30
Priority: P2 → P3
Whiteboard: [tech-debt]
Created attachment 8674399 [details] [diff] [review]
Upgrade Loop's use of eslint (to 1.6.x) and eslint-plugin-react (to 3.5.x)

I want this so that we can start using the --fix option. There's a few review comments I've had recently where if we enable a few more rules I wouldn't have to make them, those rules are currently broken, but fixable by eslint. So lets get the update, and once we finish some more of the code removals, then I'll set up a bug for the extra rules.
Attachment #8674399 - Flags: review?(edilee)
Assignee: nobody → standard8
Comment on attachment 8674399 [details] [diff] [review]
Upgrade Loop's use of eslint (to 1.6.x) and eslint-plugin-react (to 3.5.x)

+    "space-before-keywords": [2, "always"],
This rule was added in 1.4, and the code already passes that. :)

http://eslint.org/docs/rules/space-before-keywords
Attachment #8674399 - Flags: review?(edilee) → review+
https://hg.mozilla.org/mozilla-central/rev/3b9e4b34d159
Status: NEW → RESOLVED
Last Resolved: 3 years ago
status-firefox44: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla44

Updated

3 years ago
Iteration: --- → 44.2 - Oct 19
You need to log in before you can comment on or make changes to this bug.