galaxy-iuc / standards

Documentation for standards and best practices from the Galaxy IUC
http://galaxy-iuc-standards.readthedocs.io/en/latest/
6 stars 16 forks source link

xsd:boolean does not allow True. #35

Open bgruening opened 7 years ago

bgruening commented 7 years ago

Wisdom from gitter by @jmchilton and @jxtx.

xsd:boolean does not allow True Do we want to add this as recommendation?

jxtx commented 7 years ago

Would also be reasonable for Galaxy to raise a warning at parse time.

jmchilton commented 7 years ago

:+1:

Someday when I have some free time and the inclination I want to rewrite a procedure to dynamically generate a tighter XSD file from the current one that is more pedantic. One thing I will do is replace the PermissiveBoolean type with xsd:boolean - it is used dozens of times. This would be in the context of planemo linting.

xref https://github.com/galaxyproject/galaxy/blob/dev/lib/galaxy/tools/xsd/galaxy.xsd#L5117

nsoranzo commented 7 years ago

:+1:

peterjc commented 7 years ago

+1 on enforcing lower case booleans in the XSD

Cross reference https://github.com/peterjc/galaxy_blast/commit/fd7c33fac4e2358ceef4a64238b1c348d3a5459a