-
Notifications
You must be signed in to change notification settings - Fork 79
1084 qiita db #1205
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
1084 qiita db #1205
Conversation
qiita_db/commands.py
Outdated
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.
you can bring this line up, no?
|
1 minor comment 👍 |
|
Done! |
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.
There used to be four raw datas associated to this study but now there's only one? How would that happen?
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.
I removed them in populate_test_db.sql. Keeping them would require adding 3 more prep templates there, which is a lot of work...
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.
Thanks!
|
1 question, but 👍 (for the most part 💯 ). |
Built on top of #1204
Makes qiita_db to pass al tests. Still failures expected on qiita-ware and qiita-pet.