Skip to content
This repository was archived by the owner on Sep 25, 2019. It is now read-only.

Conversation

lowjoel
Copy link
Member

@lowjoel lowjoel commented Jan 19, 2016

If there's too much, commit-by-commit review should be clearer.

@lowjoel
Copy link
Member Author

lowjoel commented Jan 19, 2016

@allenwq Coverage decrease seems to be nondeterministic...

@lowjoel
Copy link
Member Author

lowjoel commented Jan 19, 2016

@fonglh once this is merged, you can deploy this already, if you intend to use the same deployment workflow as the web app.

This PR would bring the evaluator to feature completeness.

allenwq pushed a commit that referenced this pull request Jan 20, 2016
@allenwq allenwq merged commit 812f864 into Coursemology:master Jan 20, 2016
@lowjoel lowjoel deleted the evaluate-package branch January 20, 2016 09:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants