From 05d961dfe1677f097c0e7935e72a97d1cb4dfa3e Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Florian=20M=C3=BCllner?= Date: Sun, 27 Jan 2019 23:50:49 +0100 Subject: [PATCH] style: Avoid trailing commas in array destructuring When destructuring multiple return values, we often use trailing commas to indicate that there are additional elements that we are ignoring. There isn't anything inherently wrong with that, but it's a style that's too confusing for eslint - on the one hand we require a space after a comma, on the other hand we require no space before closing brackets. https://gitlab.gnome.org/GNOME/gnome-shell-extensions/merge_requests/49 --- extensions/window-list/extension.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/extensions/window-list/extension.js b/extensions/window-list/extension.js index e0ae75dd..1ff2de21 100644 --- a/extensions/window-list/extension.js +++ b/extensions/window-list/extension.js @@ -50,7 +50,7 @@ function _onMenuStateChanged(menu, isOpen) { if (isOpen) return; - let [x, y,] = global.get_pointer(); + let [x, y] = global.get_pointer(); let actor = global.stage.get_actor_at_pos(Clutter.PickMode.REACTIVE, x, y); if (Me.stateObj.someWindowListContains(actor)) actor.sync_hover();