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 Validation of Tools #127

Closed
jmchilton opened this issue Apr 15, 2015 · 4 comments
Closed

XSD Validation of Tools #127

jmchilton opened this issue Apr 15, 2015 · 4 comments

Comments

@jmchilton
Copy link
Member

WIP at https://github.com/galaxyproject/planemo/commits/lxml using https://github.com/JeanFred/Galaxy-XSD.

The approach I was thinking was:

  • Make lxml an optional dependency.
  • If it is present, add an optional linter flag that causes planemo to also validate XSD.
  • normalize the tool to expand macros and then do XSD validation on that (XSD validation with macros in place would be ... troublesome).

I don't want lxml to be a required dependency because it requires libxml et. al. which can be problematic to install on some machines.
@jmchilton jmchilton mentioned this issue Apr 15, 2015
Merged
@jmchilton
Copy link
Member Author

Made some more progress on this (now in xsd branch - https://github.com/galaxyproject/planemo/compare/xsd?expand=1) - includes WIP toward a shed_lint command #107.

@jmchilton
Copy link
Member Author

Addressed with #130

@jmchilton
Copy link
Member Author

Merged - looking forward to more PRs to Galaxy-XSD :).

@hexylena
Copy link
Member

soon.jpg :)

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
2 participants
@jmchilton @hexylena