--xsd should be the default and --disable-xsd an option

[bgruening]

I think the new xsd option should be the default behavior. @jmchilton any reason not to do so?

[jmchilton]

I think we would want to make lxml a required dependency then, which I was hesistant to do before because it required some libraries that weren't commonly installed on base systems but just the other day I did a pip install of it from pypi and got a binary manylinux wheel! It was very painless - so I'm up for this I think.

[nsoranzo]

Or even always enabled?

[bgruening]

;) I don't think there is any harm in offering a disable option for this, e..g if someone ships its own modifications to Galaxy. But the important part is to make it default :)
We can also leave out the disable option!

[peterjc]

👍 on making XSD validation the default for linting tool XML files.

[peterjc]

Enabling this has flagged all sorts of little things, proving very helpful - all of these were problems with input parameter help text using the wrong tag:

• peterjc/pico_galaxy@ca5ee85
• peterjc/pico_galaxy@26c0b04
• peterjc/pico_galaxy@50d98d1
• peterjc/galaxy_blast@f68070f
• peterjc/galaxy_blast@b31248b

[peterjc]

Another common error in my tools, using required=... rather than optional=..,

• peterjc/pico_galaxy@f7dd42e
• peterjc/galaxy_mira@0b9ce33

[peterjc]

There are a number of failures on https://github.com/galaxyproject/tools-iuc with the XSD validation, I will start filing issues...

[jmchilton]

@peterjc There is a WIP PR that I think covers all of them galaxyproject/tools-iuc#939. Might be better just to break it up and get some traction on individual parts though.