update deps and xo (segmentio#517)

docs/gatsby-browser.js

@@ -1,11 +1,11 @@
 const { NODE_ENV = 'development' } = process.env
 
-// eslint-disable-next-line import/prefer-default-export
 export const onRouteUpdate = () => {
   if (!window.analytics || typeof window.analytics.page !== 'function') {
     if (NODE_ENV === 'development') {
       console.warn('Unable to locate analytics.js')
     }
+
     return
   }
 

docs/src/components/ColorExamples.js

@@ -1,5 +1,5 @@
 import React from 'react'
-// eslint-disable-next-line import/no-extraneous-dependencies, import/no-unresolved
+// eslint-disable-next-line import/no-unresolved
 import { Pane, Heading, ThemeConsumer } from 'evergreen-ui'
 import ColorGroup from './ColorGroup'
 

docs/src/components/ColorGroup.js

@@ -1,6 +1,6 @@
 import React from 'react'
 import PropTypes from 'prop-types'
-// eslint-disable-next-line import/no-extraneous-dependencies, import/no-unresolved
+// eslint-disable-next-line import/no-unresolved
 import { Pane, Heading } from 'evergreen-ui'
 import Swatch from './Swatch'
 

docs/src/components/IconSearch.js

@@ -1,6 +1,6 @@
 import React from 'react'
 import { filter } from 'fuzzaldrin-plus'
-// eslint-disable-next-line import/no-extraneous-dependencies, import/no-unresolved
+// eslint-disable-next-line import/no-unresolved
 import { Pane, SearchInput, IconNames, Text, Icon } from 'evergreen-ui'
 
 const iconKeys = Object.keys(IconNames)

docs/src/components/Layout.js

@@ -1,4 +1,4 @@
-import React from 'react' // eslint-disable-line no-unused-vars
+import React from 'react'
 import PropTypes from 'prop-types'
 import Helmet from 'react-helmet'
 import absolutePath from '../absolutePath'

docs/src/components/Page.js

@@ -3,7 +3,7 @@ import Helmet from 'react-helmet'
 import PropTypes from 'prop-types'
 import { Link } from 'gatsby'
 import { Location, navigate } from '@reach/router'
-// eslint-disable-next-line import/no-extraneous-dependencies, import/no-unresolved
+// eslint-disable-next-line import/no-unresolved
 import { Button, IconButton, Tooltip } from 'evergreen-ui'
 import IA from '../IA'
 import DocsMDXProvider from './DocsMDXProvider'

docs/src/components/Playground.js

@@ -2,7 +2,7 @@ import React from 'react'
 import ReactDOM from 'react-dom'
 import PropTypes from 'prop-types'
 import uniqueId from 'lodash/uniqueId'
-// eslint-disable-next-line import/no-extraneous-dependencies, import/no-unresolved
+// eslint-disable-next-line import/no-unresolved
 import * as components from 'evergreen-ui'
 import Component from '@reactions/component'
 import { LiveProvider, LiveEditor, LiveError, LivePreview } from 'react-live'

docs/src/components/Swatch.js

@@ -1,6 +1,6 @@
 import React from 'react'
 import PropTypes from 'prop-types'
-// eslint-disable-next-line import/no-extraneous-dependencies, import/no-unresolved
+// eslint-disable-next-line import/no-unresolved
 import { Pane, Heading, Text, Code, Popover } from 'evergreen-ui'
 
 function uppercaseColor(color) {

docs/src/components/TextStylePreview.js

@@ -1,6 +1,6 @@
 import React from 'react'
 import PropTypes from 'prop-types'
-// eslint-disable-next-line import/no-extraneous-dependencies, import/no-unresolved
+// eslint-disable-next-line import/no-unresolved
 import { Pane } from 'evergreen-ui'
 
 export default class TextStylePreview extends React.Component {

docs/src/components/examples/AdvancedTable.js

@@ -1,6 +1,5 @@
 import React from 'react'
 import { filter } from 'fuzzaldrin-plus'
-// eslint-disable-next-line import/no-extraneous-dependencies
 import {
   Table,
   Popover,

docs/src/components/examples/VirtualTable.js

@@ -1,6 +1,5 @@
 import React, { Fragment } from 'react'
 import faker from 'faker'
-// eslint-disable-next-line import/no-extraneous-dependencies
 import {
   Table,
   Pane,
@@ -35,6 +34,7 @@ const users = range(1000)
         content: faker.random.arrayElement(randomLengthContent)
       }
     }
+
     return item
   })
 

docs/src/pages/get-started/introduction.js

@@ -1,4 +1,3 @@
-/* eslint-disable react/jsx-curly-brace-presence */
 import React from 'react'
 import Helmet from 'react-helmet'
 import TopBar from '../../components/TopBar'

package.json

@@ -77,7 +77,7 @@
     "@storybook/addon-options": "^3.4.11",
     "@storybook/addons": "^3.4.11",
     "@storybook/react": "^3.4.11",
-    "ava": "1.0.0-rc.1",
+    "ava": "^1.3.1",
     "babel-core": "7.0.0-bridge.0",
     "babel-eslint": "^10.0.1",
     "babel-plugin-add-react-displayname": "^0.0.5",
@@ -100,10 +100,10 @@
     "react-dom": "^16.5.2",
     "react-test-renderer": "^16.5.2",
     "sinon": "^7.0.0",
-    "size-limit": "^0.20.1",
+    "size-limit": "^0.21.1",
     "starwars-names": "^1.6.0",
     "url-loader": "^1.1.2",
-    "xo": "^0.23.0"
+    "xo": "^0.24.0"
   },
   "xo": {
     "parser": "babel-eslint",

src/autocomplete/stories/index.stories.js

@@ -17,6 +17,7 @@ const items = [
   if (nameA < nameB) {
     return -1
   }
+
   if (nameA > nameB) {
     return 1
   }

src/avatar/index.js

@@ -1 +1 @@
-export { default as Avatar } from './src/Avatar' // eslint-disable-line import/prefer-default-export
+export { default as Avatar } from './src/Avatar'

src/avatar/src/Avatar.js

@@ -136,6 +136,7 @@ class Avatar extends PureComponent {
     if (size <= sizeLimitOneCharacter) {
       initials = initials.substring(0, 1)
     }
+
     const colorProps = this.getColorProps()
 
     return (

src/avatar/src/utils/hash.js

@@ -10,5 +10,6 @@ export default function hashCode(s) {
     // Convert to 32bit integer
     hash &= hash
   }
+
   return Math.abs(hash)
 }

src/checkbox/index.js

@@ -1 +1 @@
-export { default as Checkbox } from './src/Checkbox' // eslint-disable-line import/prefer-default-export
+export { default as Checkbox } from './src/Checkbox'

src/combobox/index.js

@@ -1 +1 @@
-export { default as Combobox } from './src/Combobox' // eslint-disable-line import/prefer-default-export
+export { default as Combobox } from './src/Combobox'

src/combobox/src/Combobox.js

@@ -165,6 +165,7 @@ export default class Combobox extends PureComponent {
                       isOpenedByButton: false
                     })
                   }
+
                   if (e.target.value.trim() === '') {
                     // Prevent the selected item from sticking around
                     clearSelection()

src/combobox/stories/index.stories.js

@@ -13,6 +13,7 @@ const items = starWarsNames.all.sort((a, b) => {
   if (nameA < nameB) {
     return -1
   }
+
   if (nameA > nameB) {
     return 1
   }

src/corner-dialog/index.js

@@ -1 +1 @@
-export { default as CornerDialog } from './src/CornerDialog' // eslint-disable-line import/prefer-default-export
+export { default as CornerDialog } from './src/CornerDialog'

src/corner-dialog/src/CornerDialog.js

@@ -179,13 +179,15 @@ export default class CornerDialog extends PureComponent {
     if (typeof children === 'function') {
       return children({ close: this.handleClose })
     }
+
     if (typeof children === 'string') {
       return (
         <Paragraph size={400} color="muted">
           {children}
         </Paragraph>
       )
     }
+
     return children
   }
 

src/dialog/index.js

@@ -1 +1 @@
-export { default as Dialog } from './src/Dialog' // eslint-disable-line import/prefer-default-export
+export { default as Dialog } from './src/Dialog'

src/dialog/src/Dialog.js

@@ -212,9 +212,11 @@ class Dialog extends React.Component {
     if (typeof children === 'function') {
       return children({ close })
     }
+
     if (typeof children === 'string') {
       return <Paragraph>{children}</Paragraph>
     }
+
     return children
   }
 

src/dialog/stories/index.stories.js

@@ -19,6 +19,7 @@ const comboboxItems = starWarsNames.all.sort((a, b) => {
   if (nameA < nameB) {
     return -1
   }
+
   if (nameA > nameB) {
     return 1
   }

src/file-picker/index.js

@@ -1 +1 @@
-export { default as FilePicker } from './src/FilePicker' // eslint-disable-line import/prefer-default-export
+export { default as FilePicker } from './src/FilePicker'

src/file-picker/test/index.js

@@ -1,4 +1,3 @@
-/* eslint-disable import/no-extraneous-dependencies */
 import React from 'react'
 import test from 'ava'
 import render from 'react-test-renderer'

src/icon/src/Icon.js

@@ -88,6 +88,7 @@ class Icon extends PureComponent {
     if (icon == null) {
       return null
     }
+
     if (typeof icon !== 'string') {
       return icon
     }

src/image/index.js

@@ -1 +1 @@
-export { default as Image } from './src/Image' // eslint-disable-line import/prefer-default-export
+export { default as Image } from './src/Image'

src/layers/src/Pane.js

@@ -114,15 +114,19 @@ class Pane extends PureComponent {
     ) {
       return `1px solid ${theme.colors.border[borderSideProperty]}`
     }
+
     if (borderSideProperty === true) {
       return `1px solid ${theme.colors.border.default}`
     }
+
     if (borderSideProperty === false) {
       return null
     }
+
     if (Object.prototype.hasOwnProperty.call(theme.colors.border, border)) {
       return `1px solid ${theme.colors.border[border]}`
     }
+
     if (border === true) {
       return `1px solid ${theme.colors.border.default}`
     }

src/manager/index.js

@@ -1 +1 @@
-export { default as Manager } from './src/Manager' // eslint-disable-line import/prefer-default-export
+export { default as Manager } from './src/Manager'

src/menu/index.js

@@ -1 +1 @@
-export { default as Menu } from './src/Menu' // eslint-disable-line import/prefer-default-export
+export { default as Menu } from './src/Menu'

src/menu/src/MenuItem.js

@@ -92,7 +92,7 @@ class MenuItem extends React.PureComponent {
 
     if (process.env.NODE_ENV !== 'production') {
       warning(
-        typeof this.props.onClick === 'function',
+        'onClick' in this.props,
         '<Menu.Item> expects `onSelect` prop, but you passed `onClick`.'
       )
     }

src/overlay/index.js

@@ -1 +1 @@
-export { default as Overlay } from './src/Overlay' // eslint-disable-line import/prefer-default-export
+export { default as Overlay } from './src/Overlay'

src/popover/index.js

@@ -1 +1 @@
-export { default as Popover } from './src/Popover' // eslint-disable-line import/prefer-default-export
+export { default as Popover } from './src/Popover'

src/portal/index.js

@@ -1 +1 @@
-export { default as Portal } from './src/Portal' // eslint-disable-line import/prefer-default-export
+export { default as Portal } from './src/Portal'

src/portal/src/Portal.js

@@ -15,11 +15,11 @@ export default class Portal extends Component {
     if (!portalContainer) {
       portalContainer = document.createElement('div')
       portalContainer.setAttribute('evergreen-portal-container', '')
-      document.body.appendChild(portalContainer)
+      document.body.append(portalContainer)
     }
 
     this.el = document.createElement('div')
-    portalContainer.appendChild(this.el)
+    portalContainer.append(this.el)
   }
 
   componentWillUnmount() {

src/positioner/index.js

@@ -1 +1 @@
-export { default as Positioner } from './src/Positioner' // eslint-disable-line import/prefer-default-export
+export { default as Positioner } from './src/Positioner'

src/positioner/src/getPosition.js

@@ -147,6 +147,7 @@ const getTransformOrigin = ({ rect, position, dimensions, targetCenter }) => {
     /* Syntax: x-offset | y-offset */
     return `${centerX}px ${dimensions.height}px `
   }
+
   /* Syntax: x-offset | y-offset */
   return `${centerX}px 0px `
 }
@@ -268,6 +269,7 @@ function getPosition({
           rect: leftRect
         }
       }
+
       if (fitsOnRight) {
         return {
           position: Position.RIGHT,
@@ -283,6 +285,7 @@ function getPosition({
           rect: rightRect
         }
       }
+
       if (fitsOnLeft) {
         return {
           position: Position.LEFT,
@@ -357,6 +360,7 @@ function getPosition({
         rect: topRect
       }
     }
+
     if (bottomRectFitsOnBottom) {
       return {
         position: flipHorizontal(position),
@@ -372,6 +376,7 @@ function getPosition({
         rect: bottomRect
       }
     }
+
     if (topRectFitsOnTop) {
       return {
         position: flipHorizontal(position),

src/scales/src/majorScale.js

@@ -12,5 +12,6 @@ export default function majorScale(x) {
       `majorScale only accepts integers as input, instead ${x} was passed.`
     )
   }
+
   return x * 8
 }

src/scales/src/minorScale.js

@@ -12,5 +12,6 @@ export default function minorScale(x) {
       `minorScale only accepts integers as input, instead ${x} was passed.`
     )
   }
+
   return x * 4
 }

src/search-input/index.js

@@ -1 +1 @@
-export { default as SearchInput } from './src/SearchInput' // eslint-disable-line import/prefer-default-export
+export { default as SearchInput } from './src/SearchInput'

src/segmented-control/index.js

@@ -1 +1 @@
-export { default as SegmentedControl } from './src/SegmentedControl' // eslint-disable-line import/prefer-default-export
+export { default as SegmentedControl } from './src/SegmentedControl'

src/select-menu/src/Option.js

@@ -35,6 +35,7 @@ export default class Option extends PureComponent {
     if (disabled) {
       textProps.color = 'muted'
     }
+
     if (isSelected) {
       textProps.color = 'selected'
     }

src/select-menu/stories/index.stories.js

@@ -101,6 +101,7 @@ storiesOf('select-menu', module).add('SelectMenu', () => (
             } else if (selectedItemsLength > 1) {
               selectedNames = selectedItemsLength.toString() + ' selected...'
             }
+
             setState({
               selected,
               selectedNames
@@ -120,6 +121,7 @@ storiesOf('select-menu', module).add('SelectMenu', () => (
             } else if (selectedItemsLength > 1) {
               selectedNames = selectedItemsLength.toString() + ' selected...'
             }
+
             setState({ selected: selectedItems, selectedNames })
           }}
         >

src/side-sheet/index.js

@@ -1 +1 @@
-export { default as SideSheet } from './src/SideSheet' // eslint-disable-line import/prefer-default-export
+export { default as SideSheet } from './src/SideSheet'

src/side-sheet/src/SheetClose.js

@@ -120,6 +120,7 @@ const getSheetCloseClassName = position => {
       ...sharedStyles
     }).toString()
   }
+
   return sheetCloseClassNameCache[position]
 }
 

src/spinner/index.js

@@ -1 +1 @@
-export { default as Spinner } from './src/Spinner' // eslint-disable-line import/prefer-default-export
+export { default as Spinner } from './src/Spinner'