search

tibastral / contrats-francais

Contrats français
MIT License
501 stars 122 forks source link

initial commit for node.js version #34

Open barroudjo opened 5 years ago

barroudjo commented 5 years ago

Currently ruby is used to generate the pdf contracts, and given the required libraries it can only run on linux. This adds the option to use Node.js, and work thus as well on linux and windows. Could be of use to other people than me ;-).

JalilArfaoui commented 5 years ago

Maybe you could add yarn.lock or package-lock.json ?

JalilArfaoui commented 5 years ago

(And thanks for this, first of all :-) )

barroudjo commented 5 years ago

You're right, I forgot to do this. Thanks for the heads-up, and thanks for the thanks ! BTW maybe I should update the readme as well.

barroudjo commented 5 years ago

@tibastral is there something preventing this PR from being merged ? I can take care of it if there is ;-)

All the issues that houndci-bot shows are not issues actually, they're just due to its configuration, which should be updated. I'm taking a look at it.

EDIT: there's no houndci-bot configuration files so I don't see how I can configure it.

tibastral commented 5 years ago

Tu peux ajouter justement ce fichier de conf ?

Petite question : pourquoi ne pas supprimer cette dépendance qui fait que ça marche pas avec ruby plutôt que de tout refaire en node ?

barroudjo commented 5 years ago

Je vais vérifier, mais la dépendance c'est libv8, et elle me semble cruciale (plusieurs des libs de base du gemfile en ont besoin si je ne m'abuse). L'autre intérêt c'est que node.js est plus petit à installer, et que ça pourra peut être servir à d'autres d'avoir cette alternative dans un autre langage (pour customiser l'output par exemple).

tibastral commented 5 years ago

Libv8 c'est pas node justement?

Envoyé de mon iPhone

Le 6 déc. 2018 à 19:20, Jonathan notifications@github.com a écrit :

Je vais vérifier, mais la dépendance c'est libv8, et elle me semble cruciale (plusieurs des libs de base du gemfile en ont besoin si je ne m'abuse). L'autre intérêt c'est que node.js est plus petit à installer, et que ça pourra peut être servir à d'autres d'avoir cette alternative dans un autre langage (pour customiser l'output par exemple).

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

barroudjo commented 5 years ago

C'est effectivement un gem pour le moteur JavaScript V8, qui malheureusement ne compile pas sous windows... Comme une issue ouverte dessus depuis 2 ans le confirme.

Le jeu. 6 déc. 2018 à 19:38, Thibaut Assus notifications@github.com a écrit :

Libv8 c'est pas node justement?

Envoyé de mon iPhone

Le 6 déc. 2018 à 19:20, Jonathan notifications@github.com a écrit :

Je vais vérifier, mais la dépendance c'est libv8, et elle me semble cruciale (plusieurs des libs de base du gemfile en ont besoin si je ne m'abuse). L'autre intérêt c'est que node.js est plus petit à installer, et que ça pourra peut être servir à d'autres d'avoir cette alternative dans un autre langage (pour customiser l'output par exemple).

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/tibastral/contrats-francais/pull/34#issuecomment-444981370, or mute the thread https://github.com/notifications/unsubscribe-auth/AHHOgm1Z3QXkKwMQDpKmIgt4bCysbeglks5u2WRDgaJpZM4Y2GDL .

tibastral commented 5 years ago

Et si tu enlèves v8 le projet marche plus ?

Envoyé de mon iPhone

Le 6 déc. 2018 à 19:52, Jonathan notifications@github.com a écrit :

C'est effectivement un gem pour le moteur JavaScript V8, qui malheureusement ne compile pas sous windows... Comme une issue ouverte dessus depuis 2 ans le confirme.

Le jeu. 6 déc. 2018 à 19:38, Thibaut Assus notifications@github.com a écrit :

Libv8 c'est pas node justement?

Envoyé de mon iPhone

Le 6 déc. 2018 à 19:20, Jonathan notifications@github.com a écrit :

Je vais vérifier, mais la dépendance c'est libv8, et elle me semble cruciale (plusieurs des libs de base du gemfile en ont besoin si je ne m'abuse). L'autre intérêt c'est que node.js est plus petit à installer, et que ça pourra peut être servir à d'autres d'avoir cette alternative dans un autre langage (pour customiser l'output par exemple).

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/tibastral/contrats-francais/pull/34#issuecomment-444981370, or mute the thread https://github.com/notifications/unsubscribe-auth/AHHOgm1Z3QXkKwMQDpKmIgt4bCysbeglks5u2WRDgaJpZM4Y2GDL .

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

barroudjo commented 5 years ago

Tu peux pas l'enlever, c'est une dépendance de handlebars, et tu as besoin de handlebars dans ton rakefile. Donc a moins de trouver une alternative a handlebars that's it. Et comme je n'y connais rien en ruby...

Le jeu. 6 déc. 2018 à 21:13, Thibaut Assus notifications@github.com a écrit :

Et si tu enlèves v8 le projet marche plus ?

Envoyé de mon iPhone

Le 6 déc. 2018 à 19:52, Jonathan notifications@github.com a écrit :

C'est effectivement un gem pour le moteur JavaScript V8, qui malheureusement ne compile pas sous windows... Comme une issue ouverte dessus depuis 2 ans le confirme.

Le jeu. 6 déc. 2018 à 19:38, Thibaut Assus notifications@github.com a écrit :

Libv8 c'est pas node justement?

Envoyé de mon iPhone

Le 6 déc. 2018 à 19:20, Jonathan notifications@github.com a écrit :

Je vais vérifier, mais la dépendance c'est libv8, et elle me semble cruciale (plusieurs des libs de base du gemfile en ont besoin si je ne m'abuse). L'autre intérêt c'est que node.js est plus petit à installer, et que ça pourra peut être servir à d'autres d'avoir cette alternative dans un autre langage (pour customiser l'output par exemple).

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub < https://github.com/tibastral/contrats-francais/pull/34#issuecomment-444981370 , or mute the thread < https://github.com/notifications/unsubscribe-auth/AHHOgm1Z3QXkKwMQDpKmIgt4bCysbeglks5u2WRDgaJpZM4Y2GDL

.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/tibastral/contrats-francais/pull/34#issuecomment-445013927, or mute the thread https://github.com/notifications/unsubscribe-auth/AHHOgrnBcBfGZx-lt9jmVrQNBLl_69VKks5u2XpegaJpZM4Y2GDL .

tibastral commented 5 years ago

Bah si il y a https://github.com/vincent-psarga/ruby-handlebars/blob/master/README.md

Envoyé de mon iPhone

Le 6 déc. 2018 à 23:18, Jonathan notifications@github.com a écrit :

Tu peux pas l'enlever, c'est une dépendance de handlebars, et tu as besoin de handlebars dans ton rakefile. Donc a moins de trouver une alternative a handlebars that's it. Et comme je n'y connais rien en ruby...

Le jeu. 6 déc. 2018 à 21:13, Thibaut Assus notifications@github.com a écrit :

Et si tu enlèves v8 le projet marche plus ?

Envoyé de mon iPhone

Le 6 déc. 2018 à 19:52, Jonathan notifications@github.com a écrit :

C'est effectivement un gem pour le moteur JavaScript V8, qui malheureusement ne compile pas sous windows... Comme une issue ouverte dessus depuis 2 ans le confirme.

Le jeu. 6 déc. 2018 à 19:38, Thibaut Assus notifications@github.com a écrit :

Libv8 c'est pas node justement?

Envoyé de mon iPhone

Le 6 déc. 2018 à 19:20, Jonathan notifications@github.com a écrit :

Je vais vérifier, mais la dépendance c'est libv8, et elle me semble cruciale (plusieurs des libs de base du gemfile en ont besoin si je ne m'abuse). L'autre intérêt c'est que node.js est plus petit à installer, et que ça pourra peut être servir à d'autres d'avoir cette alternative dans un autre langage (pour customiser l'output par exemple).

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub < https://github.com/tibastral/contrats-francais/pull/34#issuecomment-444981370 , or mute the thread < https://github.com/notifications/unsubscribe-auth/AHHOgm1Z3QXkKwMQDpKmIgt4bCysbeglks5u2WRDgaJpZM4Y2GDL

.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/tibastral/contrats-francais/pull/34#issuecomment-445013927, or mute the thread https://github.com/notifications/unsubscribe-auth/AHHOgrnBcBfGZx-lt9jmVrQNBLl_69VKks5u2XpegaJpZM4Y2GDL .

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

barroudjo commented 5 years ago

Je peux construire l'arbre de dépendances, mais il y avait au moins une autre dépendance qui demandait libv8. Et l'alternative a handlebars que tu propose est très peu étoilée et n'a plus eu de commits depuis plusieurs années. A priori j'eviterai.

Le jeu. 6 déc. 2018 à 23:42, Thibaut Assus notifications@github.com a écrit :

Bah si il y a https://github.com/vincent-psarga/ruby-handlebars/blob/master/README.md

Envoyé de mon iPhone

Le 6 déc. 2018 à 23:18, Jonathan notifications@github.com a écrit :

Tu peux pas l'enlever, c'est une dépendance de handlebars, et tu as besoin de handlebars dans ton rakefile. Donc a moins de trouver une alternative a handlebars that's it. Et comme je n'y connais rien en ruby...

Le jeu. 6 déc. 2018 à 21:13, Thibaut Assus notifications@github.com a écrit :

Et si tu enlèves v8 le projet marche plus ?

Envoyé de mon iPhone

Le 6 déc. 2018 à 19:52, Jonathan notifications@github.com a écrit :

C'est effectivement un gem pour le moteur JavaScript V8, qui malheureusement ne compile pas sous windows... Comme une issue ouverte dessus depuis 2 ans le confirme.

Le jeu. 6 déc. 2018 à 19:38, Thibaut Assus notifications@github.com a écrit :

Libv8 c'est pas node justement?

Envoyé de mon iPhone

Le 6 déc. 2018 à 19:20, Jonathan notifications@github.com a écrit :

Je vais vérifier, mais la dépendance c'est libv8, et elle me semble cruciale (plusieurs des libs de base du gemfile en ont besoin si je ne m'abuse). L'autre intérêt c'est que node.js est plus petit à installer, et que ça pourra peut être servir à d'autres d'avoir cette alternative dans un autre langage (pour customiser l'output par exemple).

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <

https://github.com/tibastral/contrats-francais/pull/34#issuecomment-444981370

,

or mute the thread <

https://github.com/notifications/unsubscribe-auth/AHHOgm1Z3QXkKwMQDpKmIgt4bCysbeglks5u2WRDgaJpZM4Y2GDL

.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub < https://github.com/tibastral/contrats-francais/pull/34#issuecomment-445013927 , or mute the thread < https://github.com/notifications/unsubscribe-auth/AHHOgrnBcBfGZx-lt9jmVrQNBLl_69VKks5u2XpegaJpZM4Y2GDL

.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/tibastral/contrats-francais/pull/34#issuecomment-445056944, or mute the thread https://github.com/notifications/unsubscribe-auth/AHHOgi08Dp6R1nwty4EbNDEluOZma5_Sks5u2Z1VgaJpZM4Y2GDL .

tibastral commented 5 years ago

Moi si ça fait le taff je vois pas pourquoi on s'en passerait. On va être francs. Handlebars n'est pas bien compliqué et le code pourrait tenir en 10 lignes

Envoyé de mon iPhone

Le 7 déc. 2018 à 10:03, Jonathan notifications@github.com a écrit :

Je peux construire l'arbre de dépendances, mais il y avait au moins une autre dépendance qui demandait libv8. Et l'alternative a handlebars que tu propose est très peu étoilée et n'a plus eu de commits depuis plusieurs années. A priori j'eviterai.

Le jeu. 6 déc. 2018 à 23:42, Thibaut Assus notifications@github.com a écrit :

Bah si il y a https://github.com/vincent-psarga/ruby-handlebars/blob/master/README.md

Envoyé de mon iPhone

Le 6 déc. 2018 à 23:18, Jonathan notifications@github.com a écrit :

Tu peux pas l'enlever, c'est une dépendance de handlebars, et tu as besoin de handlebars dans ton rakefile. Donc a moins de trouver une alternative a handlebars that's it. Et comme je n'y connais rien en ruby...

Le jeu. 6 déc. 2018 à 21:13, Thibaut Assus notifications@github.com a écrit :

Et si tu enlèves v8 le projet marche plus ?

Envoyé de mon iPhone

Le 6 déc. 2018 à 19:52, Jonathan notifications@github.com a écrit :

C'est effectivement un gem pour le moteur JavaScript V8, qui malheureusement ne compile pas sous windows... Comme une issue ouverte dessus depuis 2 ans le confirme.

Le jeu. 6 déc. 2018 à 19:38, Thibaut Assus notifications@github.com a écrit :

Libv8 c'est pas node justement?

Envoyé de mon iPhone

Le 6 déc. 2018 à 19:20, Jonathan notifications@github.com a écrit :

Je vais vérifier, mais la dépendance c'est libv8, et elle me semble cruciale (plusieurs des libs de base du gemfile en ont besoin si je ne m'abuse). L'autre intérêt c'est que node.js est plus petit à installer, et que ça pourra peut être servir à d'autres d'avoir cette alternative dans un autre langage (pour customiser l'output par exemple).

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <

https://github.com/tibastral/contrats-francais/pull/34#issuecomment-444981370

,

or mute the thread <

https://github.com/notifications/unsubscribe-auth/AHHOgm1Z3QXkKwMQDpKmIgt4bCysbeglks5u2WRDgaJpZM4Y2GDL

.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub < https://github.com/tibastral/contrats-francais/pull/34#issuecomment-445013927 , or mute the thread < https://github.com/notifications/unsubscribe-auth/AHHOgrnBcBfGZx-lt9jmVrQNBLl_69VKks5u2XpegaJpZM4Y2GDL

.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/tibastral/contrats-francais/pull/34#issuecomment-445056944, or mute the thread https://github.com/notifications/unsubscribe-auth/AHHOgi08Dp6R1nwty4EbNDEluOZma5_Sks5u2Z1VgaJpZM4Y2GDL .

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

barroudjo commented 5 years ago

Comme je disait au début c'est pas juste handlebars qui a besoin de libv8. Enfin javascript est quand même un langage plus connu (j'ai pas dit mieux hein ! juste il y a plus de gens qui connaissent), donc ça peut permettre permettre à ces gens de customiser plus facilement la génération des contrats (marges, header, footer...). Bref ça me semble bien aussi d'avoir l'alternative.

On Fri, Dec 7, 2018 at 10:09 AM Thibaut Assus notifications@github.com wrote:

Moi si ça fait le taff je vois pas pourquoi on s'en passerait. On va être francs. Handlebars n'est pas bien compliqué et le code pourrait tenir en 10 lignes

Envoyé de mon iPhone

Le 7 déc. 2018 à 10:03, Jonathan notifications@github.com a écrit :

Je peux construire l'arbre de dépendances, mais il y avait au moins une autre dépendance qui demandait libv8. Et l'alternative a handlebars que tu propose est très peu étoilée et n'a plus eu de commits depuis plusieurs années. A priori j'eviterai.

Le jeu. 6 déc. 2018 à 23:42, Thibaut Assus notifications@github.com a écrit :

Bah si il y a

https://github.com/vincent-psarga/ruby-handlebars/blob/master/README.md

Envoyé de mon iPhone

Le 6 déc. 2018 à 23:18, Jonathan notifications@github.com a écrit :

Tu peux pas l'enlever, c'est une dépendance de handlebars, et tu as besoin de handlebars dans ton rakefile. Donc a moins de trouver une alternative a handlebars that's it. Et comme je n'y connais rien en ruby...

Le jeu. 6 déc. 2018 à 21:13, Thibaut Assus notifications@github.com a écrit :

Et si tu enlèves v8 le projet marche plus ?

Envoyé de mon iPhone

Le 6 déc. 2018 à 19:52, Jonathan notifications@github.com a écrit :

C'est effectivement un gem pour le moteur JavaScript V8, qui malheureusement ne compile pas sous windows... Comme une issue ouverte dessus depuis 2 ans le confirme.

Le jeu. 6 déc. 2018 à 19:38, Thibaut Assus < notifications@github.com> a écrit :

Libv8 c'est pas node justement?

Envoyé de mon iPhone

Le 6 déc. 2018 à 19:20, Jonathan notifications@github.com a écrit :

Je vais vérifier, mais la dépendance c'est libv8, et elle me semble cruciale (plusieurs des libs de base du gemfile en ont besoin si je ne m'abuse). L'autre intérêt c'est que node.js est plus petit à installer, et que ça pourra peut être servir à d'autres d'avoir cette alternative dans un autre langage (pour customiser l'output par exemple).

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <

https://github.com/tibastral/contrats-francais/pull/34#issuecomment-444981370

,

or mute the thread <

https://github.com/notifications/unsubscribe-auth/AHHOgm1Z3QXkKwMQDpKmIgt4bCysbeglks5u2WRDgaJpZM4Y2GDL

.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <

https://github.com/tibastral/contrats-francais/pull/34#issuecomment-445013927

,

or mute the thread <

https://github.com/notifications/unsubscribe-auth/AHHOgrnBcBfGZx-lt9jmVrQNBLl_69VKks5u2XpegaJpZM4Y2GDL

.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub < https://github.com/tibastral/contrats-francais/pull/34#issuecomment-445056944 , or mute the thread < https://github.com/notifications/unsubscribe-auth/AHHOgi08Dp6R1nwty4EbNDEluOZma5_Sks5u2Z1VgaJpZM4Y2GDL

.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/tibastral/contrats-francais/pull/34#issuecomment-445168689, or mute the thread https://github.com/notifications/unsubscribe-auth/AHHOgjfn6VjUsYRuMCYvnTTi-Ggr3FlGks5u2jAugaJpZM4Y2GDL .

tibastral commented 5 years ago

EN fait pour tout dire, on avait migré vers JS il y a quelques temps, mais ensuite, on a re-migré vers ruby notamment parce que j'avais pas envie de maintenir du js. Après, si tu es chaud pour faire une version elm, go for it, je mergerai avec joie, mais js je suis vraiment pas fan.