Skip to content

Update eslint to error with vue-no-new-non-primitive rule

Paul Slaughter requested to merge ps-eslint-vue-no-new-non-primitive-error into master

What does this MR do and why?

This MR actually enables the custom eslint rule vue-no-new-non-primitive-in-template which was added in this MR gitlab-org/frontend/eslint-plugin!43 (merged).

Creating new non-primitive objects in a template can cause reactive issues (see relevant comment).

Please note that there's some trivial fixes to some trivial violations. To ease with this MR's review, we opt for just adding the eslint-disable / eslint-enable lines around violations. We'll follow this up with an epic to resolve existing violations.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Paul Gascou-Vaillancourt

Merge request reports