[ESLint] Fix ESLint issues in devtools/client/webconsole/test/browser_webconsole_live_filtering_of_message_types.js

RESOLVED DUPLICATE of bug 1177324

Status

()

Firefox
Developer Tools: Console
P3
normal
RESOLVED DUPLICATE of bug 1177324
a year ago
a year ago

People

(Reporter: linclark, Assigned: a.l.e, Mentored)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [btpp-backlog])

Attachments

(1 attachment)

(Reporter)

Description

a year ago
If you haven’t contributed to Firefox before, follow the steps here to set up your environment: https://developer.mozilla.org/en-US/docs/Tools/Contributing#Getting_set_up

Then, automatically configure ESLint to work with the Firefox specific rules by following the instructions here: https://wiki.mozilla.org/DevTools/CodingStandards

Then you can see the issues that need to be fixed by running

> eslint --no-ignore devtools/client/webconsole/test/browser_webconsole_live_filtering_of_message_types.js
(Reporter)

Updated

a year ago
Blocks: 1256948
Priority: -- → P3
(Reporter)

Updated

a year ago
Whiteboard: [btpp-backlog]
(Assignee)

Comment 1

a year ago
Created attachment 8747435 [details] [diff] [review]
0001-Bug-1256895-ESLint-Fix-ESLint-issues-in-devtools-cli.patch

greetings from the zurich mozilla hackaton!
Attachment #8747435 - Flags: review?(lclark)
https://treeherder.mozilla.org/#/jobs?repo=try&revision=a0263376aaf2
Assignee: nobody → ale.comp_06
(Reporter)

Comment 3

a year ago
Comment on attachment 8747435 [details] [diff] [review]
0001-Bug-1256895-ESLint-Fix-ESLint-issues-in-devtools-cli.patch

Review of attachment 8747435 [details] [diff] [review]:
-----------------------------------------------------------------

This looks good, thanks!
Attachment #8747435 - Flags: review?(lclark) → review+
(Reporter)

Comment 4

a year ago
Before we commit this, could you update the patch's commit message?

Change this line:

> Bug 1256895 - [ESLint] Fix ESLint issues in devtools/client/webconsole/test/browser_webconsole_live_filtering_of_message_types.js

To: 

> Bug 1256895 - [ESLint] Fix ESLint issues in devtools/client/webconsole/test/browser_webconsole_live_filtering_of_message_types.js. r=linclark


And for the user, unfortunately I don't think that we support having multiple users, so please delete the " + richard eigenmann" part.
Changing the commit message can be done locally, you don't need to use Janitor for that. Do you guys want me to do that or do you want to finish this?


(In reply to Lin Clark [:linclark] from comment #4)
> And for the user, unfortunately I don't think that we support having
> multiple users, so please delete the " + richard eigenmann" part.

As discussed with Lin through IRC, this won't be necessary since we don't have strict ties to other systems there. We definitely want to credit both of you :)

Small note: in case the "New contributors to Firefox" list that comes out with every Firefox release looks for those names, it will display exactly that. If it doesn't and looks at the Bugzilla name, then maybe just Ale will be credited in the post.
Flags: needinfo?(ale.comp_06)
Thanks a lot Ale and Richard for this patch!

Unfortunately we weren't able to land it in time, and bug 1177324 fixed the same problem again (bug 1271084 auto-fixed the space to the generator function).

If you liked contributing to Firefox, and would like to try fixing other Firefox / Devtools bugs, there are great resources to find good first bugs, such as:

http://www.joshmatthews.net/bugsahoy/

and

http://firefox-dev.tools/
Status: NEW → RESOLVED
Last Resolved: a year ago
Flags: needinfo?(ale.comp_06)
Resolution: --- → DUPLICATE
Duplicate of bug: 1177324
You need to log in before you can comment on or make changes to this bug.