search

Litarvan / lightdm-webkit-theme-litarvan

Litarvan's LightDM HTML Theme
BSD 3-Clause "New" or "Revised" License
707 stars 73 forks source link

style: add eslint and prettier #183

Open Aakodal opened 2 years ago

Aakodal commented 2 years ago

Litarvan: feel free to change rules you disapprove.

EDIT: this PR is left aside until Litarvan decides to refactor the whole project (not my words, but his).

Aakodal commented 2 years ago

The results of the Prettier config should be similar as similar as before, without introducing a lot of changes in a simple PR.

Oh well that didn't change a lot xD I changed tabWidth to 4 since I committed your suggestion (4 and not 2 because it was 4 before this PR), but it looks like there are still more than 20k lines changed, mostly caused by code inconsistency.

In that case, shouldn't I revert to tabs? Even if final rule is up to Litarvan, I'd recommand them since everyone can configure how they look, and they are literally meant for indentation.