-
Notifications
You must be signed in to change notification settings - Fork 57
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
reboot Lint & format #1173
reboot Lint & format #1173
Conversation
sq add doc
It's copied from extract-file-from-node-module script
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, merci pour ce taf @seballot
Je me demandais d’où venaient toutes ces règles php en plus de celles de PSR12 et Symfony, j’espère que tu n'as pas passé un temps monstre a les pondre...
Sinon j'ai une petite tension sur le fait de rajouter les configs pour vscode (pourquoi vscode et pas les autres IDE? ;) ), je trouve que cela contribue a renforcer son hégémonie, une mention dans la doc des plugins recommandés et d'un exemple de conf m'aurais paru plus neutre, mais en vrai c'est pas le plus important, on pourra toujours faire marche arrière quand il sera avéré que vscode espionne ses usager.es et nourrit des IA un peu sales (QUOI c'est déjà le cas, mais que fait la police ??!!!! ).
Bref je peux merger, mais je rant un peu quand même..
ah ouai t'en a vraiment après vscode ah ah :) Tu peux rajouter de la doc pour neovim si tu veux non? mais je trouve que ça serait dommage de priver des nouveaux dev utilisant vscode de la config toute prête niveau linter. On peut mettre pour "vs codium" si tu préfères ah ah Sinon question, les test qui fails sur la PR il faut en tenir compte ou pas? ou ça fail tout le temps? |
pour les règles c'es chatGpt qui me les a fourni, et puis j'en ai ajusté 4 ou 5 selon mes préférences |
Pour les tests, ca fail tout le temps pour les tests, un 403 sur le curl d'installation, pas la peine de regarder, ca sera corrigé avec la branche nouvel installateur. Et préconiser vs codium plutot que vscode, c'est déja un pas dans la bonne direction (la config neovim viendra aussi, mais ya un peu une config par utilisateurice de neovim donc ca va etre tendu :D ). |
feat mrflos ;)
ok merci ! j'ai fait la modif pour vs codium :) |
J'ai retravaillé le linting, surtout niveau php car PSR12 ne donnait pas assez de règles de formattage
J'ai remplacé stylelint par prettier, et comme ça on peut l'utiliser sur d'autres fichiers
J'ai rajouté un peu de doc aussi