turn off array destructuring linter rule

Change-Id: I829458cde3b979030f305a71bc943246a388d042
Reviewed-on: https://gerrit.instructure.com/137475
Reviewed-by: Derek Bender <djbender@instructure.com>
Tested-by: Jenkins
Product-Review: Jeremy Neander <jneander@instructure.com>
QA-Review: Jeremy Neander <jneander@instructure.com>

.eslintrc.js

@@ -47,6 +47,7 @@ module.exports = {
     "no-unused-vars": [2, { "argsIgnorePattern": "^_"}],
     "one-var": ["error", { initialized: "never" }], // allow `let foo, bar` but not `let foo=1, bar=2`
     "object-curly-spacing": [0],
+    "prefer-destructuring": ["error", {"object": true, "array": false}],
     "padded-blocks": [0], // so we can have space between the define([... and the callback
     "semi": [0],
     "import/no-extraneous-dependencies": [0], // allows 'i18n!webzip_exports' and 'compiled/foo/bar'