diff --git a/package.json b/package.json index da80461bde0..041176c675e 100644 --- a/package.json +++ b/package.json @@ -48,7 +48,7 @@ "@typescript-eslint/parser": "1.13.0", "@zendeskgarden/css-bedrock": "7.0.28", "@zendeskgarden/css-variables": "6.3.7", - "@zendeskgarden/eslint-config": "8.0.0", + "@zendeskgarden/eslint-config": "9.0.0", "@zendeskgarden/stylelint-config": "10.0.0", "@zendeskgarden/svg-icons": "5.3.1", "babel-core": "7.0.0-bridge.0", diff --git a/packages/autocomplete/src/elements/Multiselect.js b/packages/autocomplete/src/elements/Multiselect.js index f8b0f764227..a5ba84231ce 100644 --- a/packages/autocomplete/src/elements/Multiselect.js +++ b/packages/autocomplete/src/elements/Multiselect.js @@ -44,7 +44,7 @@ const StyledInput = styled(Input)` } ${props => - props.selectedValues.length !== 0 && // eslint-disable-line + props.selectedValues.length !== 0 && (!props.isFocused || props.tagFocusedKey !== undefined) && !props.isOpen && !props.placeholder && diff --git a/packages/buttons/src/elements/ButtonGroup.js b/packages/buttons/src/elements/ButtonGroup.js index c6d29b68b74..cf215c822df 100644 --- a/packages/buttons/src/elements/ButtonGroup.js +++ b/packages/buttons/src/elements/ButtonGroup.js @@ -93,7 +93,7 @@ export default class ButtonGroup extends ControlledComponent { }; render() { - const { children, ...otherProps } = this.props; // eslint-disable-line no-unused-vars + const { children, ...otherProps } = this.props; const { focusedKey, selectedKey, id } = this.getControlledState(); return ( diff --git a/packages/checkboxes/src/elements/Checkbox.js b/packages/checkboxes/src/elements/Checkbox.js index e082547cb72..3e13bd8548b 100644 --- a/packages/checkboxes/src/elements/Checkbox.js +++ b/packages/checkboxes/src/elements/Checkbox.js @@ -52,7 +52,7 @@ export default class Checkbox extends ControlledComponent { getHintProps, keyboardFocused } = {}) => { - const { children, wrapperProps, id, ...checkboxInputProps } = this.props; // eslint-disable-line no-unused-vars + const { children, wrapperProps, id, ...checkboxInputProps } = this.props; /** * Due to us applying keyboard-only focus events to 2 separate elements (the input and label) diff --git a/packages/dropdowns/src/Fields/Label.tsx b/packages/dropdowns/src/Fields/Label.tsx index 33dace2b550..e17e4b030ab 100644 --- a/packages/dropdowns/src/Fields/Label.tsx +++ b/packages/dropdowns/src/Fields/Label.tsx @@ -1,4 +1,3 @@ -/* eslint-disable react/prop-types */ /** * Copyright Zendesk, Inc. * diff --git a/packages/loaders/src/Skeleton.js b/packages/loaders/src/Skeleton.js index ecc1d576521..56480e348f0 100644 --- a/packages/loaders/src/Skeleton.js +++ b/packages/loaders/src/Skeleton.js @@ -41,7 +41,6 @@ const skeletonRtlAnimation = keyframes` } `; -/* eslint-disable */ const StyledSkeleton = styled.div.attrs({ 'data-garden-id': COMPONENT_ID, 'data-garden-version': PACKAGE_VERSION diff --git a/packages/radios/src/elements/Radio.js b/packages/radios/src/elements/Radio.js index 80780683b54..5c6f2409258 100644 --- a/packages/radios/src/elements/Radio.js +++ b/packages/radios/src/elements/Radio.js @@ -52,7 +52,7 @@ export default class Radio extends ControlledComponent { getHintProps, keyboardFocused } = {}) => { - const { children, wrapperProps, id, ...checkboxInputProps } = this.props; // eslint-disable-line no-unused-vars + const { children, wrapperProps, id, ...checkboxInputProps } = this.props; /** * Due to us applying keyboard-only focus events to 2 separate elements (the input and label) diff --git a/packages/select/src/elements/Select.js b/packages/select/src/elements/Select.js index 01d8894a057..3f3846f3b32 100644 --- a/packages/select/src/elements/Select.js +++ b/packages/select/src/elements/Select.js @@ -203,7 +203,7 @@ export default class Select extends ControlledComponent { eventsEnabled, popperModifiers, children, - onChange, // eslint-disable-line no-unused-vars + onChange, small, selectRef, disabled: selectDisabled, diff --git a/packages/tables/src/views/Table.js b/packages/tables/src/views/Table.js index 699df547109..646bc3828e2 100644 --- a/packages/tables/src/views/Table.js +++ b/packages/tables/src/views/Table.js @@ -23,7 +23,6 @@ const SIZE = { LARGE: 'large' }; -/* eslint-disable indent */ const retrieveSrollableStyling = props => { if (!props.scrollable) { return ''; diff --git a/packages/tabs/src/elements/Tabs.js b/packages/tabs/src/elements/Tabs.js index b72ea810e85..611a40ed590 100644 --- a/packages/tabs/src/elements/Tabs.js +++ b/packages/tabs/src/elements/Tabs.js @@ -119,7 +119,7 @@ export default class Tabs extends ControlledComponent { } // Don't want to duplicate tabProps in the TabPanel - const { tabProps, ...other } = child.props; // eslint-disable-line no-unused-vars + const { tabProps, ...other } = child.props; return cloneElement( child, diff --git a/packages/toggles/src/elements/Toggle.js b/packages/toggles/src/elements/Toggle.js index 5d4566959cc..f9798117bfb 100644 --- a/packages/toggles/src/elements/Toggle.js +++ b/packages/toggles/src/elements/Toggle.js @@ -52,7 +52,7 @@ export default class Toggle extends ControlledComponent { getHintProps, keyboardFocused } = {}) => { - const { children, wrapperProps, id, ...checkboxInputProps } = this.props; // eslint-disable-line no-unused-vars + const { children, wrapperProps, id, ...checkboxInputProps } = this.props; /** * Due to us applying keyboard-only focus events to 2 separate elements (the input and label) diff --git a/yarn.lock b/yarn.lock index 4ce05b1cd15..8f441090487 100644 --- a/yarn.lock +++ b/yarn.lock @@ -2798,10 +2798,10 @@ resolved "https://registry.yarnpkg.com/@zendeskgarden/css-variables/-/css-variables-6.3.7.tgz#63c3955d28811b525c8f5e80912e883eeff0d82c" integrity sha512-YDDRpdUTaGLN5jfBvMOR5AMXhA36+CQ5bb9lJG90FvH00uxIqh5Q4KgD1M7bdALcPhbMq24wAp9ITy/1LJtnDw== -"@zendeskgarden/eslint-config@8.0.0": - version "8.0.0" - resolved "https://registry.yarnpkg.com/@zendeskgarden/eslint-config/-/eslint-config-8.0.0.tgz#e81d09d511c79dbecef02a0a6442ca32f7caaf68" - integrity sha512-ASutc2nGI364sTsef5f5M45+MFTw+n/guF1RmPtL5BIEGKdmnIlXG7NI3d4iSVotp/3+QGDlTe83B+cabv795Q== +"@zendeskgarden/eslint-config@9.0.0": + version "9.0.0" + resolved "https://registry.yarnpkg.com/@zendeskgarden/eslint-config/-/eslint-config-9.0.0.tgz#f506b86d2b53dc860a58a7f0d8b6303b2858cfbb" + integrity sha512-8u3ZD/cvnkWKXHTTOuNI8RFSJMolAlfORJ5fxzCLXRWQc0pNKzEEUENnScNSsufXysOWQiiY6DY7d9jH/vgmRg== "@zendeskgarden/stylelint-config@10.0.0": version "10.0.0"