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
Instrument model #1484
Instrument model #1484
Conversation
This is ready for review: @biocore/qiita-dev |
"(instrumet model wasn't checked): %s" % ( | ||
', '.join(nvp))) | ||
if nvim: | ||
error_msgs.append("These samples have ado not have a valid " |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ado -> do
reminded me of a song though ... 🎼
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oh and have is before the not
.
I think this looks good overall, I would just suggest adding a note to the ChangeLog about this new required column for submissions to EBI. |
👍 once @ElDeveloper comments are addressed. There will be some merge conflicts with my PR, but it is way easier to solve them in my branch. |
Ready to go! |
👍 @ElDeveloper can you review/merge? |
This adds the instrument model field as a required field for EBI submission. Most of the changes are due to fixing tests. Notable changes: changed the template documentation page, and travis.yml to remove two lines and make it one (suggestion by @josenavas in previous request and found how to do it).