Ajout de fichiers pour gérer des dépendances de développement #51
Conversation
OK, je vais remplacer par |
julienw
force-pushed
the
use-dev-dependencies
branch
from
1c1c16d to
a198d9d
Compare
|
Je fais juste un build puis j'upload sur addons.mozilla.org qui s'occupe de la signature. |
julienw
force-pushed
the
use-dev-dependencies
branch
3 times, most recently
from
9e6157f to
612d692
Compare
| "webExt": { | ||
| "ignoreFiles": [ | ||
| "*.md", | ||
| "package*.json" | ||
| ] | ||
| } |
There was a problem hiding this comment.
J'ai découvert ça, c'est bien pratique pour avoir des options utilisées pour toutes les commandes et avoir du coup des commandes plus claires :-)
package.json
Outdated
| "lint": "prettier --check .", | ||
| "lint-fix": "prettier --check -w .", | ||
| "start": "web-ext run", | ||
| "build": "web-ext build --overwrite-dest" |
There was a problem hiding this comment.
j'ai laissé le --overwrite-dest sur la ligne de commande plutôt que le mettre dans l'objet de config ci-dessous, car je me suis dit que tu voudrais peut-être pas de ce comportement si tu exécutes web-ext directement. Mais si tu préfères, je peux le déplacer.
There was a problem hiding this comment.
Je ne mettrai pas du tout cette option perso, c'est mieux d'être sûr de ce qu'on fait quand on build.
There was a problem hiding this comment.
Comme tu veux, c'est toi qui buildes après tout ;-)
julienw
force-pushed
the
use-dev-dependencies
branch
3 times, most recently
from
9828586 to
92429f8
Compare
|
@dunglas je veux bien un second check, et surtout que tu essaies les différents trucs chez toi pour dire si ça te va :-) |
| "author": "Kévin Dunglas <[email protected]>", | ||
| "license": " AGPL-3.0-only", | ||
| "scripts": { | ||
| "lint": "prettier --check .", |
There was a problem hiding this comment.
| "lint": "prettier --check .", | |
| "lint": "prettier --check . && web-ext lint", |
There was a problem hiding this comment.
J'avais fait ça d'abord, mais je ne suis pas convaincu que ça fonctionne sous Windows, et puis si prettier échoue tu n'as pas le résultat de web-ext lint...
There was a problem hiding this comment.
Dis moi tout de même si tu préfères ça, et je le fais :-)
There was a problem hiding this comment.
T'as raison, ça ne marche pas sous Windows... Je hais l'écosystème JS :D
Du coup je me demande si ces scripts et ces deps ont un réel intérêt comparé à juste indiquer dans le README de lancer npx prettier --write . && npx web-ext lint ?
There was a problem hiding this comment.
pour le lint JS avec prettier, je trouve ça un peu chiant à taper en entier ;-)
(bon c'est aussi que mon vim n'est pas configuré pour l'exécuter car dans mon projet "normal" on utilise eslint qui lance lui prettier... alors je veux bien plaider coupable)
pour le web-ext je n'avais pas ajouté de script parce que je suis d'accord sur le peu d'intérêt...
There was a problem hiding this comment.
Utilise ça dans ce cas peut être? https://prettier.io/docs/en/watching-files.html
There was a problem hiding this comment.
J'utilise vim aussi mais il est config pour auto-prettier xD
Sinon je suis + pour rester simple et utiliser seulement les outils avec npx, garder le côté léger et fonctionnel du projet
| - name: Lint the web extension | ||
| run: npx web-ext lint | ||
|
|
There was a problem hiding this comment.
| - name: Lint the web extension | |
| run: npx web-ext lint |
There was a problem hiding this comment.
Petit comportement intéressant de npx: il utilise le package local au projet s'il est présent :-)
donc ça utilise bien la version installée dans le projet.
(j'ai appris ça lorsque j'ai fait cette PR, je trouve que yarn est beaucoup plus logique pour ça finalement...)
There was a problem hiding this comment.
Oui je me suis déjà fait avoir sur d'autres projets par ce comportement (pas très intuitif je trouve).
julienw
force-pushed
the
use-dev-dependencies
branch
from
92429f8 to
adecc48
Compare
|
Je ferai une autre pr demain avec juste un contributing.md, on pourra comparer. |
|
J'ai hate d'avoir cette tache et une meilleure intégration de Prettier :-) |
Fixes #49
J'ai vérifié:
node_modulesest automatiquement ignoré parweb-ext. En revanche package.json, yarn.lock ou même README.md sont dans le paquet généré. Est-ce un souci ?Je ne suis pas convaincu par l'usage de yarn. Je l'ai utilisé par habitude, mais ça ajoute une étape lors de l'installation du projet... Vous en pensez quoi ?