Fix no-unused-vars issues in the Pocket extension

RESOLVED FIXED in Firefox 52

Status

()

defect
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: standard8, Assigned: standard8)

Tracking

Trunk
Firefox 52
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox52 fixed)

Details

Attachments

(1 attachment)

Assignee

Description

3 years ago
There's a few bits of dead code in the Pocket extension that I'd like to clean up as part of the ongoing no-unused-vars code.

To test this, I added the following to browser/.eslintrc.js and did `./mach eslint browser/extensions/pocket`:

  "rules": {
    "no-unused-vars": ["error", {
      "vars": "local",
      "varsIgnorePattern": "^Cc|Ci|Cu|Cr|EXPORTED_SYMBOLS",
      "args": "none",
    }]
  }

We'll be enabling the rule for all of browser/ in bug 1312407.
Comment hidden (mozreview-request)
Assignee

Comment 2

3 years ago
Shane/Mike, how do we ensure these changes get back into the Pocket repo (not sure if there's other changes that have landed as well).
Flags: needinfo?(mozilla)
Flags: needinfo?(mixedpuppy)
Assignee

Updated

3 years ago
Blocks: 1315617
Comment on attachment 8808083 [details]
Bug 1315615 - Fix no-unused-vars issues in the Pocket extension.

https://reviewboard.mozilla.org/r/91006/#review90808
Attachment #8808083 - Flags: review?(jaws) → review+
I'll get a pull request together for them.
Flags: needinfo?(mozilla)
Flags: needinfo?(mixedpuppy)

Comment 5

3 years ago
Pushed by mbanner@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/fd11584b7ab0
Fix no-unused-vars issues in the Pocket extension. r=jaws

Comment 6

3 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/fd11584b7ab0
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 52

Comment 7

3 years ago
(In reply to Mike Kaply [:mkaply] from comment #4)
> I'll get a pull request together for them.

Looks like this is https://github.com/mozilla-partners/pocket/pull/11 , but there was no traction? Can we just merge this?
Flags: needinfo?(mozilla)
Yes, merge it
Flags: needinfo?(mozilla)
You need to log in before you can comment on or make changes to this bug.