ESLint: Enable ignoreRestSiblings option for no-unused-vars rule

Author: tyxla

package-lock.json

@@ -18,13 +18,11 @@ export default [
 		},
 		supports,
 		migrate: ( oldAttributes ) => {
-			/* eslint-disable no-unused-vars */
 			const {
 				singleCommentLabel,
 				multipleCommentsLabel,
 				...newAttributes
 			} = oldAttributes;
-			/* eslint-enable no-unused-vars */
 			return newAttributes;
 		},
 		isEligible: ( { multipleCommentsLabel, singleCommentLabel } ) =>

packages/block-library/src/comments-title/deprecated.js

@@ -98,13 +98,8 @@ class KeyboardAvoidingView extends Component {
 	}
 
 	render() {
-		const {
-			children,
-			enabled,
-			keyboardVerticalOffset, // eslint-disable-line no-unused-vars
-			style,
-			...props
-		} = this.props;
+		const { children, enabled, keyboardVerticalOffset, style, ...props } =
+			this.props;
 
 		let finalStyle = style;
 		if ( Platform.OS === 'ios' ) {

packages/components/src/mobile/bottom-sheet/keyboard-avoiding-view.native.js

@@ -9,7 +9,6 @@ export const useNavigationTreeNodes = () => {
 	const getNode = ( key ) => nodes[ key ];
 
 	const addNode = ( key, value ) => {
-		// eslint-disable-next-line no-unused-vars
 		const { children, ...newNode } = value;
 		return setNodes( ( original ) => ( {
 			...original,
@@ -19,7 +18,6 @@ export const useNavigationTreeNodes = () => {
 
 	const removeNode = ( key ) => {
 		return setNodes( ( original ) => {
-			// eslint-disable-next-line no-unused-vars
 			const { [ key ]: removedNode, ...remainingNodes } = original;
 			return remainingNodes;
 		} );

packages/components/src/navigation/use-navigation-tree-nodes.js

@@ -319,13 +319,7 @@ class NativeEditorProvider extends Component {
 	}
 
 	render() {
-		const {
-			children,
-			post, // eslint-disable-line no-unused-vars
-			capabilities,
-			settings,
-			...props
-		} = this.props;
+		const { children, post, capabilities, settings, ...props } = this.props;
 		const editorSettings = this.getEditorSettings( settings, capabilities );
 
 		return (

packages/editor/src/components/provider/index.native.js

@@ -52,7 +52,7 @@ module.exports = {
 		'no-unreachable': 'error',
 		'no-unsafe-negation': 'error',
 		'no-unused-expressions': 'error',
-		'no-unused-vars': 'error',
+		'no-unused-vars': [ 'error', { ignoreRestSiblings: true } ],
 		'no-useless-return': 'error',
 		'no-whitespace-before-property': 'error',
 		'no-with': 'error',

packages/eslint-plugin/configs/es5.js

@@ -9,7 +9,7 @@ module.exports = {
 		'no-trailing-spaces': 'error',
 		'no-undef': 'error',
 		'no-unused-expressions': 'error',
-		'no-unused-vars': 'error',
+		'no-unused-vars': [ 'error', { ignoreRestSiblings: true } ],
 		'one-var': [ 'error', 'always' ],
 		quotes: [ 'error', 'single' ],
 		'wrap-iife': [ 'error', 'any' ],

packages/eslint-plugin/configs/jshint.js

@@ -214,7 +214,6 @@ class AztecView extends Component {
 	}
 
 	render() {
-		// eslint-disable-next-line no-unused-vars
 		const { onActiveFormatsChange, ...otherProps } = this.props;
 		// `style` has to be destructured separately, without `otherProps`, because of:
 		// https://github.com/WordPress/gutenberg/issues/23611

packages/react-native-aztec/src/AztecView.js

@@ -36,7 +36,6 @@ const registerGutenberg = ( {
 		constructor( props ) {
 			super( props );
 
-			// eslint-disable-next-line no-unused-vars
 			const { rootTag, ...parentProps } = this.props;
 
 			// Setup locale.