Skip to content

Remove eslint-plugin-html

Lukas 'ai-pi' Eipert requested to merge leipert-remove-eslint-html-plugin into master

What does this MR do?

With a quick check it was easily discoverable that the plugin is currently not used.

node_modules/.bin/eslint --print-config app/ | grep html

It has been added with 8050d835 but removed with eb336d71 for vue files and completely removed with 3e9bb4ad.

As this plugin is unused and actively hindering an eslint upgrade, we should remove it.

Screenshots

Does this MR meet the acceptance criteria?

Conformity

Performance and Testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • Label as security and @ mention @gitlab-com/gl-security/appsec
  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • Security reports checked/validated by a reviewer from the AppSec team

Merge request reports