Skip to content
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

Merged
merged 11 commits into from
Jun 19, 2024
Merged

reboot Lint & format #1173

merged 11 commits into from
Jun 19, 2024

Conversation

seballot
Copy link
Contributor

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

@seballot seballot requested a review from mrflos June 19, 2024 08:51
Copy link
Contributor

@mrflos mrflos left a 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..

@seballot
Copy link
Contributor Author

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?

@seballot
Copy link
Contributor Author

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

@mrflos
Copy link
Contributor

mrflos commented Jun 19, 2024

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 ).

@seballot
Copy link
Contributor Author

ok merci !

j'ai fait la modif pour vs codium :)

@mrflos mrflos merged commit 4080e71 into doryphore-dev Jun 19, 2024
2 of 4 checks passed
@mrflos mrflos deleted the lint-format branch June 19, 2024 11:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants