Enforce not having parens around arguments of single argument arrow f…

…unctions.
m1rp · Dec 2, 2016 · 71629d8 · 71629d8
1 parent a319971
commit 71629d8
Show file tree

Hide file tree

Showing 5 changed files with 10 additions and 9 deletions.
diff --git a/.eslintrc.js b/.eslintrc.js
@@ -84,6 +84,7 @@ module.exports = {
             "named": "never",
             "asyncArrow": "always",
         }],
+        "arrow-parens": ["error", "as-needed"],
     },
     "settings": {
         "react": {

diff --git a/src/common/summarize-profile.js b/src/common/summarize-profile.js
@@ -244,7 +244,7 @@ export function calculateRollingSummaries(profile, threadCategories, segmentCoun
   return profile.threads.map((thread, threadIndex) => {
     const categories = threadCategories[threadIndex];
 
-    return times(segmentCount, (segmentIndex) => {
+    return times(segmentCount, segmentIndex => {
       let samplesInRange = 0;
       const samples = {};
 

diff --git a/src/content/async-storage.js b/src/content/async-storage.js
@@ -71,7 +71,7 @@ export function getStore(dbName) {
   function getItem(key) {
     return new Promise((resolve, reject) => {
       let req;
-      withStore('readonly', (store) => {
+      withStore('readonly', store => {
         store.transaction.oncomplete = function onComplete() {
           let value = req.result;
           if (value === undefined) {
@@ -89,7 +89,7 @@ export function getStore(dbName) {
 
   function setItem(key, value) {
     return new Promise((resolve, reject) => {
-      withStore('readwrite', (store) => {
+      withStore('readwrite', store => {
         store.transaction.oncomplete = resolve;
         const req = store.put(value, key);
         req.onerror = function setItemOnError() {
@@ -101,7 +101,7 @@ export function getStore(dbName) {
 
   function removeItem(key) {
     return new Promise((resolve, reject) => {
-      withStore('readwrite', (store) => {
+      withStore('readwrite', store => {
         store.transaction.oncomplete = resolve;
         const req = store.delete(key);
         req.onerror = function removeItemOnError() {
@@ -113,7 +113,7 @@ export function getStore(dbName) {
 
   function clear() {
     return new Promise((resolve, reject) => {
-      withStore('readwrite', (store) => {
+      withStore('readwrite', store => {
         store.transaction.oncomplete = resolve;
         const req = store.clear();
         req.onerror = function clearOnError() {
@@ -126,7 +126,7 @@ export function getStore(dbName) {
   function length() {
     return new Promise((resolve, reject) => {
       let req;
-      withStore('readonly', (store) => {
+      withStore('readonly', store => {
         store.transaction.oncomplete = function onComplete() {
           resolve(req.result);
         };
@@ -146,7 +146,7 @@ export function getStore(dbName) {
       }
 
       let req;
-      withStore('readonly', (store) => {
+      withStore('readonly', store => {
         store.transaction.oncomplete = function onComplete() {
           const cursor = req.result;
           resolve(cursor ? cursor.key : null);

diff --git a/src/content/components/SummarizeLineGraph.js b/src/content/components/SummarizeLineGraph.js
@@ -84,7 +84,7 @@ class SummarizeLineGraph extends Component {
 
   render() {
     return (
-      <div className='summarize-line-graph' ref={(el) => { this.el = el; }}>
+      <div className='summarize-line-graph' ref={el => { this.el = el; }}>
         {
           this.state && this.props && this.props.rollingSummary
             ? <svg

diff --git a/src/content/components/TreeView.js b/src/content/components/TreeView.js
@@ -221,7 +221,7 @@ class TreeView extends Component {
 
     const selected = this.props.selectedNodeId;
     const visibleRows = this._getAllVisibleRows(this.props);
-    const selectedRowIndex = visibleRows.findIndex((nodeId) => nodeId === selected);
+    const selectedRowIndex = visibleRows.findIndex(nodeId => nodeId === selected);
 
     if (selectedRowIndex === -1) {
       this._select(visibleRows[0]);