-
Notifications
You must be signed in to change notification settings - Fork 13
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
Ajoute une validation JSDoc #1194
Conversation
✅ Deploy Preview for stylo-docs canceled.
|
✅ Deploy Preview for stylo-dev ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Trop bien 👌🏻 |
Hm, ce que tu demandes c'est la validation typescript… dans le code ? Sinon j'ai pas compris. |
Oui mais sans utiliser TypeScript 😄 |
fe6be87
to
f53bb42
Compare
J'ai ajouté un commit qui tente une vérification typescript. Bon, je ne suis pas vraiment sûr si ça doit être eslint qui vérifie ça ou si c'est ok d'ajouter le module |
C'était plus par curiosité, autant avancer par étape, le plugin Je pense qu'on peut fusionner avec ton premier commit 👍🏻 |
48a0322
to
f53bb42
Compare
Oui carrément. J'ai enlevé le dernier commit. |
Ça nous permettra d'être plus rigoureux sur la documentation des fonctions. Y'a 64 erreurs avec la documentation existante :-) (les corrections viennent de
eslint --fix
).Un chouette article : https://schof.co/writing-jsdoc-for-react-components/