Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

xsd:boolean does not allow True. #35

Open
bgruening opened this issue Jan 3, 2017 · 4 comments
Open

xsd:boolean does not allow True. #35

bgruening opened this issue Jan 3, 2017 · 4 comments

Comments

@bgruening
Copy link
Member

Wisdom from gitter by @jmchilton and @jxtx.

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

jxtx commented Jan 3, 2017

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

@jmchilton
Copy link
Contributor

👍

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
Copy link
Member

nsoranzo commented Jun 2, 2017

👍

@peterjc
Copy link
Contributor

peterjc commented Jun 2, 2017

+1 on enforcing lower case booleans in the XSD

Cross reference peterjc/galaxy_blast@fd7c33f

peterjc added a commit to peterjc/galaxy_mira that referenced this issue Jun 2, 2017
@peterjc
XML booleans should be lower case
1d7d466 
Cross reference galaxy-iuc/standards#35

Planemo and the Galaxy XSD currently uses permissive booleans in
many places so this is currently not spotted by planemo lint.
peterjc added a commit to peterjc/pico_galaxy that referenced this issue Jun 5, 2017
@peterjc
XML booleans should be lower case
191123e 
Cross reference galaxy-iuc/standards#35

Planemo and the Galaxy XSD currently uses permissive booleans in
many places so this is currently not spotted by planemo lint.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@peterjc @jxtx @jmchilton @bgruening @nsoranzo