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
Created attachment 8747435 [details] [diff] [review] 0001-Bug-1256895-ESLint-Fix-ESLint-issues-in-devtools-cli.patch greetings from the zurich mozilla hackaton!
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!
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.
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/