Skip to content

Show eslint error for v-html usages

Dheeraj Joshi requested to merge djadmin-eslint-v-html-error into master

What does this MR do and why?

This should prompt an error instead of a warning for any v-html usages in the codebase.

  • We use eslint-plugin-vue with the plugin:vue/recommended configuration which should have this as warning by default

  • To err on the safe side, it's important to show an error instead of a warning to prevent against any potential errors.

It got removed with !63918 (diffs), possibly because there were no violations found during the migration.

Screenshots or screen recordings

before after
image image

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

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 Dheeraj Joshi

Merge request reports