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

Ajoute une validation JSDoc #1194

Merged
merged 1 commit into from
Jan 24, 2025
Merged

Ajoute une validation JSDoc #1194

merged 1 commit into from
Jan 24, 2025

Conversation

thom4parisot
Copy link
Member

@thom4parisot thom4parisot commented Jan 23, 2025

Ç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/

Copy link

netlify bot commented Jan 23, 2025

Deploy Preview for stylo-docs canceled.

Name Link
🔨 Latest commit f53bb42
🔍 Latest deploy log https://app.netlify.com/sites/stylo-docs/deploys/67939043d6744900081904b2

Copy link

netlify bot commented Jan 23, 2025

Deploy Preview for stylo-dev ready!

Name Link
🔨 Latest commit f53bb42
🔍 Latest deploy log https://app.netlify.com/sites/stylo-dev/deploys/67939043e3b66c00088a0172
😎 Deploy Preview https://deploy-preview-1194--stylo-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@ggrossetie
Copy link
Collaborator

Trop bien 👌🏻
Est ce qu'il y a des outils qui valident la documentation avec l'utilisation ? (i.e., j'utilise un type qui est incorrect par rapport à la jsdoc)

@thom4parisot
Copy link
Member Author

Hm, ce que tu demandes c'est la validation typescript… dans le code ? Sinon j'ai pas compris.

@ggrossetie
Copy link
Collaborator

Oui mais sans utiliser TypeScript 😄

@thom4parisot
Copy link
Member Author

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 typescript (il n'a pas l'air aware de certains trucs liés à vite).

@ggrossetie
Copy link
Collaborator

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 typescript (il n'a pas l'air aware de certains trucs liés à vite).

C'était plus par curiosité, autant avancer par étape, le plugin eslint est déjà une très bonne première étape. Quand on aura plus de jsdoc en place ça vaudra surement le coup d'aller plus loin dans la validation.

Je pense qu'on peut fusionner avec ton premier commit 👍🏻

@thom4parisot
Copy link
Member Author

Oui carrément. J'ai enlevé le dernier commit.

@thom4parisot thom4parisot merged commit 00cd285 into master Jan 24, 2025
9 checks passed
@thom4parisot thom4parisot deleted the feat/eslint-jsdoc branch January 24, 2025 13:06
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