Skip to content

BUG: Fix bug in moi.init #1123

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

Merged
merged 2 commits into from
Apr 29, 2015
Merged

BUG: Fix bug in moi.init #1123

merged 2 commits into from
Apr 29, 2015

Conversation

ElDeveloper
Copy link
Contributor

The position of the group_id argument was moved and the rest of the code
was not updated to reflect this. This change set moves group_id so it is
the first argument.

The position of the group_id argument was moved and the rest of the code
was not updated to reflect this. This change set moves group_id so it is
the first argument.
@ElDeveloper
Copy link
Contributor Author

This problem would cause the analysis waiting page to be blank.

@ElDeveloper ElDeveloper added this to the Alpha 0.1 milestone Apr 29, 2015
@wasade
Copy link
Contributor

wasade commented Apr 29, 2015

I'd say to open a PR upstream, but I know your take on moi...

@ElDeveloper
Copy link
Contributor Author

@wasade, I went over there first, but I saw this: https://github.com/biocore/mustached-octo-ironman/pull/19/files ... I've added a note in the code 🍺

@coveralls
Copy link

Coverage Status

Coverage remained the same at 79.43% when pulling 13fa06e on ElDeveloper:moi-bug into 33e5404 on biocore:master.

@ElDeveloper
Copy link
Contributor Author

Tests are passing but it seems like coveralls has choked, anyone wants to have a look? This is blocking for the release.

antgonza added a commit that referenced this pull request Apr 29, 2015
@antgonza antgonza merged commit 7ed5b1f into qiita-spots:master Apr 29, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants