[enh] #18: check arguments retrieval is done with env var.

YunoHost / package_linter

Linter for YunoHost applications packages

https://yunohost.org/#/packaging_apps

GNU Affero General Public License v3.0

17 stars 13 forks source link

[enh] #18: check arguments retrieval is done with env var. #22

Closed M5oul closed 7 years ago

M5oul commented 7 years ago

'$1' arg retrieval is now deprecated.

maniackcrudelis commented 7 years ago

Bonne idée dans le principe, mais à mon avis, limiter ton code à 10 arguments et aux 30 premières lignes finira par te poser des problèmes. C'est certes beaucoup plus simple. Mais pour assurer l'efficacité du test, je pense qu'il aurait été préférable de chercher la présence des arguments dans le script install en fonction du manifest. (Mais c'est plus compliqué je pense)

M5oul commented 7 years ago

mais à mon avis, limiter ton code à 10 arguments et aux 30 premières lignes finira par te poser des problèmes.

Oui, il y a des faux positifs, par exemple dans rainloop, il y a des fonctions qui ont $1.

Pour le script d’installation, c’est bien de vérifier la correspondance avec le manifest. En effet, ça pourrait être une amélioration.

Par contre, pour les autres scripts, tels l’upgrade et le restore récupèrent des arguments via ce moyen.