Merge pull request #1025 from plotly/renovate/prettier-2.x

Update dependency prettier to v2

Author: dmt0

.prettierrc

@@ -1,6 +1,5 @@
 {
-    "singleQuote": true,
-    "bracketSpacing": false,
-    "trailingComma": "es5",
-    "printWidth": 100
+  "singleQuote": true,
+  "bracketSpacing": false,
+  "printWidth": 100
 }

package.json

@@ -86,7 +86,7 @@
     "postcss-combine-duplicated-selectors": "8.1.0",
     "postcss-custom-properties": "8.0.11",
     "postcss-remove-root": "0.0.2",
-    "prettier": "1.19.1",
+    "prettier": "2.0.2",
     "react": "16.13.1",
     "react-ace": "7.0.5",
     "react-dom": "16.13.1",

src/DefaultEditor.js

@@ -33,13 +33,13 @@ class DefaultEditor extends Component {
   }
 
   hasTransforms() {
-    return this.context.fullData.some(d => TRANSFORMABLE_TRACES.includes(d.type));
+    return this.context.fullData.some((d) => TRANSFORMABLE_TRACES.includes(d.type));
   }
 
   hasAxes() {
     return (
       Object.keys(this.context.fullLayout._subplots).filter(
-        type =>
+        (type) =>
           !['cartesian', 'mapbox'].includes(type) &&
           this.context.fullLayout._subplots[type].length > 0
       ).length > 0
@@ -63,15 +63,15 @@ class DefaultEditor extends Component {
   }
 
   hasColorbars() {
-    return this.context.fullData.some(d => traceHasColorbar({}, d));
+    return this.context.fullData.some((d) => traceHasColorbar({}, d));
   }
 
   hasLegend() {
-    return this.context.fullData.some(t => t.showlegend !== undefined); // eslint-disable-line no-undefined
+    return this.context.fullData.some((t) => t.showlegend !== undefined); // eslint-disable-line no-undefined
   }
 
   hasMaps() {
-    return this.context.fullData.some(d =>
+    return this.context.fullData.some((d) =>
       [...TRACE_TO_AXIS.geo, ...TRACE_TO_AXIS.mapbox].includes(d.type)
     );
   }

src/EditorControls.js

@@ -31,7 +31,7 @@ class EditorControls extends Component {
   constructor(props, context) {
     super(props, context);
 
-    this.localize = key => localizeString(this.props.dictionaries || {}, this.props.locale, key);
+    this.localize = (key) => localizeString(this.props.dictionaries || {}, this.props.locale, key);
 
     // we only need to compute this once.
     if (this.props.plotly) {
@@ -107,7 +107,7 @@ class EditorControls extends Component {
               props = shamefullyCreateSplitStyleProps(graphDiv, attr, traceIndex, splitTraceGroup);
             }
 
-            props.forEach(p => {
+            props.forEach((p) => {
               if (value !== void 0) {
                 p.set(value);
               }
@@ -429,8 +429,8 @@ EditorControls.defaultProps = {
   showFieldTooltips: false,
   locale: 'en',
   traceTypesConfig: {
-    categories: _ => categoryLayout(_),
-    traces: _ => traceTypes(_),
+    categories: (_) => categoryLayout(_),
+    traces: (_) => traceTypes(_),
     complex: true,
   },
   fontOptions: DEFAULT_FONTS,

src/__stories__/index.js

@@ -98,7 +98,7 @@ const panelsToTest = {
   funnelarea: ['GraphCreatePanel', 'StyleTracesPanel'],
 };
 
-window.URL.createObjectURL = function() {
+window.URL.createObjectURL = function () {
   return null;
 };
 
@@ -128,10 +128,10 @@ const panelFixture = (Panel, group, name, figure, customConfig) => {
 
 let stories = storiesOf('Panels', module);
 
-Object.keys(mocks).forEach(m => {
+Object.keys(mocks).forEach((m) => {
   const selectedPanels = panelsToTest[m] ? panelsToTest[m] : Object.keys(panels);
 
-  selectedPanels.forEach(p => {
+  selectedPanels.forEach((p) => {
     const words = p.split(/(?=[A-Z])/);
     const panelGroup = words[0];
     const panelName = words.slice(1, -1).join(' ');

src/__tests__/syntax-test.js

@@ -3,16 +3,16 @@ import fs from 'fs';
 import glob from 'glob';
 
 const BLACK_LIST = ['fdescribe', 'fit', 'xdescribe', 'xit', 'it\\.only', 'describe\\.only'];
-const REGEXS = BLACK_LIST.map(token => new RegExp(`^\\s*${token}\\(.*`));
+const REGEXS = BLACK_LIST.map((token) => new RegExp(`^\\s*${token}\\(.*`));
 
 describe('Syntax and test validation', () => {
   describe(`ensures ${BLACK_LIST} is not present in tests`, () => {
     const files = glob.sync('!(node_modules|examples)/**/__tests__/*.js');
-    files.forEach(file =>
+    files.forEach((file) =>
       it(`checks ${file} for test checks`, () => {
         const code = fs.readFileSync(file, {encoding: 'utf-8'});
-        code.split('\n').forEach(line => {
-          expect(REGEXS.some(re => re.test(line))).toBe(false);
+        code.split('\n').forEach((line) => {
+          expect(REGEXS.some((re) => re.test(line))).toBe(false);
         });
       })
     );

src/components/PanelMenuWrapper.js

@@ -9,7 +9,7 @@ class PanelsWithSidebar extends Component {
     super(props);
 
     const opts = this.computeMenuOptions(props);
-    const firstSidebarGroup = opts.filter(o => o.panels)[0];
+    const firstSidebarGroup = opts.filter((o) => o.panels)[0];
 
     this.state = {
       group: firstSidebarGroup.name,
@@ -53,7 +53,7 @@ class PanelsWithSidebar extends Component {
     let groupIndex;
     const childrenArray = sortMenu(React.Children.toArray(children), menuPanelOrder);
 
-    childrenArray.forEach(child => {
+    childrenArray.forEach((child) => {
       if (!child) {
         return;
       }

src/components/containers/MapboxLayersAccordion.js

@@ -30,7 +30,7 @@ class MapboxLayersAccordion extends Component {
 
     const addAction = {
       label: _('Layer'),
-      handler: context => {
+      handler: (context) => {
         const {fullContainer, updateContainer} = context;
         if (updateContainer) {
           const mapboxLayerIndex = Array.isArray(fullContainer.layers)

src/components/containers/PlotlyFold.js

@@ -63,7 +63,7 @@ export class Fold extends Component {
       canDelete && typeof deleteContainer === 'function' ? (
         <div
           className="fold__top__delete js-fold__delete"
-          onClick={e => {
+          onClick={(e) => {
             e.stopPropagation();
             deleteContainer(foldInfo);
           }}
@@ -76,7 +76,7 @@ export class Fold extends Component {
       <div className="fold__top__moving-controls">
         <span
           className={`fold__top__moving-controls--up${canMoveUp ? '' : '--disabled'}`}
-          onClick={e => {
+          onClick={(e) => {
             // prevents fold toggle to happen when clicking on moving arrow controls
             e.stopPropagation();
 
@@ -92,7 +92,7 @@ export class Fold extends Component {
         </span>
         <span
           className={`fold__top__moving-controls--down${canMoveDown ? '' : '--disabled'}`}
-          onClick={e => {
+          onClick={(e) => {
             // prevents fold toggle to happen when clicking on moving arrow controls
             e.stopPropagation();
             if (canMoveDown) {
@@ -184,7 +184,7 @@ class PlotlyFold extends Fold {
   determineVisibility(nextProps, nextContext) {
     this.foldVisible = false;
 
-    React.Children.forEach(nextProps.children, child => {
+    React.Children.forEach(nextProps.children, (child) => {
       if (!child || this.foldVisible) {
         return;
       }

src/components/containers/PlotlyPanel.js

@@ -47,7 +47,7 @@ export class Panel extends Component {
 
   toggleFolds() {
     const {individualFoldStates} = this.state;
-    const hasOpen = individualFoldStates.length > 0 && individualFoldStates.some(s => s !== true);
+    const hasOpen = individualFoldStates.length > 0 && individualFoldStates.some((s) => s !== true);
     this.setState({
       individualFoldStates: individualFoldStates.map(() => hasOpen),
     });
@@ -61,7 +61,7 @@ export class Panel extends Component {
     // to get proper number of child folds and initialize component state
     let numFolds = 0;
 
-    React.Children.forEach(this.props.children, child => {
+    React.Children.forEach(this.props.children, (child) => {
       if (((child && child.type && child.type.plotly_editor_traits) || {}).foldable) {
         numFolds++;
       }
@@ -114,7 +114,7 @@ export class Panel extends Component {
           addAction={this.props.addAction}
           allowCollapse={this.props.showExpandCollapse && individualFoldStates.length > 1}
           toggleFolds={this.toggleFolds}
-          hasOpen={individualFoldStates.some(s => s === false)}
+          hasOpen={individualFoldStates.some((s) => s === false)}
         />
         <div className={bem('panel', 'content')}>{newChildren}</div>
       </div>