search

wegue-oss / wegue

Template and components for webmapping applications with OpenLayers and Vue.js
BSD 2-Clause "Simplified" License
97 stars 41 forks source link

Transition to new ESLint configuration #379

Open chrismayer opened 6 months ago

chrismayer commented 6 months ago

A new format for ESLint configuration files is supported since 2022 and will be the default one in version 9. Transitioning to this new configuration file format should be done but once again when our main dependencies will export those kind of files. This is already the case for eslint-plugin-vue but not for eslint-config-standard (well it is already if you checkout the master branch on GitHub but not if you install the official NPM package).
Making the transition will force us to get rid of @vue/eslint-config-standard which is used automatically by vue-cli and first versions of create-vue. What it basically does is configure ESLint to use vue rules and standardJS ones.
Newer versions of create-vue don't use it anymore but don't include standardJS anymore neither so I think we could easily depart from @vue/eslint-config-standard but what about standardJS? Should we keep it? I personally think we should as I like this style guide but please give me your opinion on this one!

This issue is derived from #377 and the text above was initially provided by @sronveaux .

sronveaux commented 1 month ago

Just a follow-up on this one, it seems like StandardJS is kind of abandoned and that a lot of people transit to neostandard which tries to align with current StandardJS with minor changes (mainly due to plugins not yet transited to the new configuration format) which are made by the community through an open governance. It is still in an early stage but I think we should keep an eye on it if we decide to keep aligned to StandardJS.

We should however wait for V1 to be out I think and also perhaps for Wegue to be transited to Vite if possible so eslint-plugin-vue could be removed easily.

Don't hesitate to give your opinion about this one!