Reconsider checking the apps scripts with ShellCheck

(Related discussion : add shellcheck config by OniriCorpe · Pull Request #220 · YunoHost/example_ynh · GitHub )

(Personal opinion from a few days ago on the chat

yyyyah heuarg i should get more familiar with shellcheck, i was kind of like “meh it’s gonna spit out so many errors” and i’m like “uuuugh yet another file” but not really a good argument

and then i’m like zhgfmvflfv we should really have packaging v3 to drastically reduce the amount of bash code again rather than micro-managing the current syntax :expressionless:

but maybe displaying the shellcheck results in the linter could be gud, should give this a try

Basically yes but if we suddently add this in the linter this may be overwhelming idk, should give this a try

2 Likes