geraldinepascal / FROGS-wrappers

Galaxy wrappers for FROGS
2 stars 12 forks source link

Small review #10

Closed lecorguille closed 3 years ago

lecorguille commented 3 years ago

Hi here,

We wish to install Frogs on metabarcoding.usegalaxy.fr (a subdomain of usegalaxy.fr) to replace galaxy4metab.sb-roscoff.fr

But according to IUC Best Practices, for security reasons:

All Cheetah variables for text parameters, input and output files must be single-quoted, e.g. '${var_name}'.

Since I was having a look at the wrappers, I didn't resist to enhance a little with macros and a new tool versionning.

Up to you to accept this PR but you are disagree with some part, I can remove some of the extra commits and just keep the single-quoting.

A+

(ping @loraine-gueguen)