NPM Resource Changed PR-6.0-dev Pending

User tests: Successful: Unsuccessful:

avatar C-Lodder
C-Lodder
19 Sep 2025

Summary of Changes

This migrates linting from ESLint to Biome, for 2 simple reasons:

  1. Performance - Biome is at least 15x faster than ESLint due to the fact it's written in Rust and it multi-threadded.
  2. Less dependencies - ESLint requires multiple plugins to work with languages and frameworks/libraries, whereas Biome supports them out of the box.

As part of this PR, I've also run the built-in formatter to try and use concise coding standards across the board.

To-Do

  • Fix some remaining CSS lint issues
  • Migrate CSS ordering rules
  • Remove lint:css task from package.json
  • Update Github action

@Fedik @dgrammatiko Would be nice to get your thoughts

avatar C-Lodder C-Lodder - open - 19 Sep 2025
avatar C-Lodder C-Lodder - change - 19 Sep 2025
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 19 Sep 2025
Category JavaScript Administration com_media NPM Change
avatar C-Lodder C-Lodder - change - 19 Sep 2025
Labels Added: NPM Resource Changed PR-6.0-dev
avatar joomla-cms-bot joomla-cms-bot - change - 19 Sep 2025
Category JavaScript Administration com_media NPM Change Repository JavaScript Administration com_media NPM Change
avatar C-Lodder C-Lodder - change - 19 Sep 2025
The description was changed
avatar C-Lodder C-Lodder - edited - 19 Sep 2025
avatar laoneo
laoneo - comment - 19 Sep 2025

Did you add a line length? Would probably make sense to increase it, to have less changes in this pr.

avatar C-Lodder
C-Lodder - comment - 19 Sep 2025

@laoneo The default is 80, which I believe is the same as ESLint.

Happy to increase this if you'd like a specific value, although 80 is generally considered a good number.

avatar laoneo
laoneo - comment - 19 Sep 2025

If I remember correctly we were going in the past with 150. But can't find any evidence right now. Perhaps @wilsonge can shed some light here. Personally, I find 80 to close.

avatar laoneo
laoneo - comment - 19 Sep 2025

These here are definitely incorrect.

Add a Comment

Login with GitHub to post a comment