refactor: migrate stories next to components

[xballoy]

I migrated all the stories next to their component.

• For some of them I had to add a devDependencies because some component use other component in their stories. I don't know if that's an issue but it's working as is.
• I removed the demo on how to use the form, it had some external dependencies and wasn't really something related to storybook, but really a demo.

Not related to this PR but some quick improvements:

• I removed the pre-commit package because it was overriding husky and no linter was run on pre-commit.
• I removed the *.ts from the TypeScript linter as we weren't linting the tsx files. That broke the linting but I fixed it in the PR.
• I excluded **/*.stories.tsx from Sonar (in addition of .js)

Fixes #726

[arnaudforaison]

A quoi cela sert ? Je ne me souviens plus

[xballoy]

C'est pour les import des markdown dans les stories en TS

[arnaudforaison]

Ah oui les readme

[xballoy]

Oui, c'était une erreur eslint, je n'avais même pas fait gaffe 😅

…

-------- Original Message --------

On 19 Oct 2020, 18:34, Arnaud Foraison wrote: @arnaudforaison commented on this pull request. --------------------------------------------------------------- In [packages/Form/Input/pass/src/PassInput.stories.js](#731 (comment)): > @@ -66,7 +63,7 @@ const PassStory = () => ( const PassInputStory = () => ( <form className="af-form" name="myform"> <div> - <PassInput + <PassInputHOC Ok je vois ça ferait une confusion entre l'export par default et l'export nommé — You are receiving this because you authored the thread. Reply to this email directly, [view it on GitHub](#731 (comment)), or [unsubscribe](https://github.com/notifications/unsubscribe-auth/AAFHRYJJVMF4ENCYI3CEIGTSLRTBFANCNFSM4SV6UEZA).

[samuel-gomez-axa]

j'ai réinstallé et lancer le storybook, ça fonctionne nickel , bravo :)
J'ai juste eu des warnings dans la console de lancement (sur toutes les stories) :

Et il y a une anomalie graphique sur la démo sur le composant /story/help--html :

il ne devrait pas y avoir de fond bleu, probablement du à la restructuration du composant

[xballoy]

Bizarre ce warning, surtout que ça build vraiment... Je regarderais demain (et pour le help aussi).

…

-------- Original Message --------

On 19 Oct 2020, 18:50, Samuel Gomez wrote: j'ai réinstallé et lancer le storybook, ça fonctionne nickel , bravo :) J'ai juste eu des warnings dans la console de lancement (sur toutes les stories) : [Capture2](https://user-images.githubusercontent.com/43414022/96481164-77f75380-123b-11eb-8c14-8109f19b068c.JPG) Et il y a une anomalie graphique sur la démo sur le composant /story/help--html : [Capture](https://user-images.githubusercontent.com/43414022/96481383-bee54900-123b-11eb-95f6-9b258f0bc373.JPG) il ne devrait pas y avoir de fond bleu, probablement du à la restructuration du composant — You are receiving this because you authored the thread. Reply to this email directly, [view it on GitHub](#731 (comment)), or [unsubscribe](https://github.com/notifications/unsubscribe-auth/AAFHRYMZF2KPPZ25XW5QIXDSLRU5BANCNFSM4SV6UEZA).

[xballoy]

@samuel-gomez-axa J'ai corrigé le problème sur le popover, c'était un problème d'import (j'ai importé le composant pas le container).
Par contre je ne trouve rien pour les warnings...

[xballoy]

J'ai simplifié l'option --ignore du compile-js.js pour que l'on ne compile plus que les fichiers sources (pas les specs, ni les stories).
J'ai aussi ajouté .src/__snapshots__ pour ne pas copier les snapshots mais cela ne fonctionne pas (c'était déjà comme ça) à cause de ce bug (babel/babel#11394) qui a été corrigé hier. Dans une prochaine version de babel ca devrait fonctionner.

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!

0 Bugs
0 Vulnerabilities (and 0 Security Hotspots to review)
0 Code Smells

100.0% Coverage
0.0% Duplication