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.

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

[18.05] Added missing level fatal_oom to xsd #6341

Merged

Conversation

bernt-matthias
Copy link
Contributor

fixes #6340

@nsoranzo nsoranzo changed the title added missing level fatal_oom to xsd [18.05] Added missing level fatal_oom to xsd Jun 15, 2018
@nsoranzo
Copy link
Member

Can you also add it to the <exit_code> (around line 4866) and <regex> documentation (around line 4934), please?

@bernt-matthias
Copy link
Contributor Author

I hope I found all the places where docs were missing. Oddly some examples for using oom were already present.

Will the PR find its way to dev and planemo?

@nsoranzo
Copy link
Member

Will the PR find its way to dev and planemo?

@bernt-matthias Yes, thanks for working on this!

@bernt-matthias
Copy link
Contributor Author

Welcome. I use the oom+resubmission feature quite a lot. See also #6338 :)

@jmchilton jmchilton merged commit a2ff5ef into galaxyproject:release_18.05 Jun 16, 2018
@jmchilton
Copy link
Member

Thanks @bernt-matthias for the update, and thanks for the review @nsoranzo !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants