Git submodule pour le Javascript du picker d'emojis

Darks commented

2023-06-12 09:22:55 +02:00

Owner

Depuis qu'on a inclus l'emoji-picker dans le code de l'éditeur de Markdown, le dépot a plus de Javascript que de Python…

Bon c'est pas tant pour les stats que pour la propreté, mais il faudrait setup un git submodule pour ce bout de code.

Le dépot existe déjà (https://gitea.planet-casio.com/Eragon/emoji-picker-element), y'a plus qu'à faire la config.

Avantage notable : pas besoin de copier/coller le code à chaque mise à jour, un git pull suffit.

Avantage annexe : les stats des langages utilisés par la v5 devraient revenir à des niveaux représentatifs

Depuis qu'on a inclus l'emoji-picker dans le code de l'éditeur de Markdown, le dépot a plus de Javascript que de Python… Bon c'est pas tant pour les stats que pour la propreté, mais il faudrait setup un git submodule pour ce bout de code. Le dépot existe déjà (https://gitea.planet-casio.com/Eragon/emoji-picker-element), y'a plus qu'à faire la config. Avantage notable : pas besoin de copier/coller le code à chaque mise à jour, un git pull suffit. Avantage annexe : les stats des langages utilisés par la v5 devraient revenir à des niveaux représentatifs

Darks added the

performance

label 2023-06-12 09:22:55 +02:00

Lephenixnoir commented

2023-06-12 09:41:31 +02:00

Owner

Note personnelle : bonne idée le sous-module à mon avis, comme ça quand on clone --recursive PCv5 tout est versionné de façon centralisée. Ça mitige le casse-tête d'avoir plein de dépôts différents (pour lesquels y'a des problèmes de synchronisation etc, cf. gint/fxSDK)

Eragon commented

2023-06-12 09:45:45 +02:00

Member

Le truc qui me dérange avec le sous-module c'est qu'on est sensé build le picker en fait.
Ajouter en sous-module c'est pas suffisant, ok ça sort le picker du code de la v5 et c'est très bien. Mais le désavantage c'est qu'il faut le build et copier les fichiers json et js dans le dossier actuel des scripts.
On peut facilement ajouter ça au makefile, mais du coup je me demande si ça vaut le coup de créer un dossier /submodules et d'y placer le sous-module git. Ou si on préfère faire une release dans le repo du picker et on télécharge juste les fichiers pre-construit dans la release.

Le truc qui me dérange avec le sous-module c'est qu'on est sensé build le picker en fait. Ajouter en sous-module c'est pas suffisant, ok ça sort le picker du code de la v5 et c'est très bien. Mais le désavantage c'est qu'il faut le build et copier les fichiers json et js dans le dossier actuel des scripts. On peut facilement ajouter ça au makefile, mais du coup je me demande si ça vaut le coup de créer un dossier `/submodules` et d'y placer le sous-module git. Ou si on préfère faire une release dans le repo du picker et on télécharge juste les fichiers pre-construit dans la release.

Lephenixnoir commented

2023-06-12 09:47:29 +02:00

Owner

C'est plus simple d'automatiser le make durant le déploiement que d'automatiser le téléchargement du zip durant la release non ?

C'est plus simple d'automatiser le `make` durant le déploiement que d'automatiser le téléchargement du zip durant la release non ?

Eragon commented

2023-06-12 10:10:58 +02:00

Member

Pour moi ça semble équivalent.