Resolve "Migrate to ESLint Flat Config"
What does this MR do and why?
This MR upgrades Eslint package from 8.57.0
to 9.15.0
(latest).
It also changes eslint configuration to flat config (Used npx @eslint/migrate-config .eslintrc.yml
for auto migration and then followed some more instructions from https://vueschool.io/articles/vuejs-tutorials/upgrading-eslint-from-v8-to-v9-in-vue-js/). Also referenced main GitLab repo's migration here.
Why?
- Flat config consolidates all eslint related configuration into one file. Easier for maintenance.
- We have not been upgrading Eslint and now with the new Eslint, we have to use the flat configuration as well.
How to set up and validate locally
Numbered steps to set up and validate the change are strongly suggested.
-
Configure a local GitLab Docs environment: https://gitlab.com/gitlab-org/technical-writing-group/gitlab-docs-hugo/-/blob/main/doc/setup.md. -
Run make lint-frontend
. Make sure there are no console errors. -
Make sure we can build local environment normally and also confirm there are no console errors on the website inspect.
Merge request acceptance checklist
This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.
-
I have evaluated the MR acceptance checklist for this merge request.
Closes #146 (closed)
Edited by Hiru Fernando