Changes made since Xavier's last review - #21

Closed

martynjames wants to merge 18 commits into

pre_cleanupfrom

Commits on Jan 30, 2015

Merge pull request #13 from edx-solutions/mjevtic/analytics-fixes
Martyn James
committed
0c5718c
Clean up and reorganization
Martyn James
committed
ad86b84

Commits on Feb 2, 2015

Fixed unit tests to run on the newest version of workbench
marjev
committed
0887699

Commits on Feb 3, 2015

Updated requirements file
marjev
committed
1b6dbb6
Split long lines into two
marjev
committed
7920340
Added the YAML file for Travis; Added coverage to requirements.txt; Described how someone can get test coverage in README file
marjev
committed
39c446a
Merge pull request #15 from edx-solutions/mjevtic/SOL-299
Martyn James
committed
86f99fc

Commits on Feb 4, 2015

pep8 and pylint should be supported and runable on Travis; changed the test coverage report to only include google_drive
marjev
committed
8f39a0b
Merge pull request #16 from edx-solutions/mjevtic/SOL-299
Martyn James
committed
403dc44
Ignore duplicate-code in pylint - it is failing on 5 common lines between the 2 XBlock components herein - better to have a common pattern than to seprate out these lines. Also use virtual display …
Martyn James
committed
51e1002
Update README to show travis project status
Martyn James
committed
75c50bf
Merge pull request #14 from edx-solutions/clean_up
Martyn James
committed
90399e7

Commits on Feb 6, 2015

Changes as a result of feedback
Martyn James
committed
0a83943
Merge pull request #20 from edx-solutions/mjames/Feedback_Updates
Martyn James
committed
9d76471

Commits on Feb 19, 2015

Added integration tests for studio views; Code clean-up;
marjev
committed
64c7b7a
Feedback inspired changes

Martyn James
authored and
marjev
committed
57ccc55
Merge pull request #22 from edx-solutions/mjames/feedback_updates
marjev
committed
e9233c4
Merge pull request #23 from edx-solutions/mjevtic/PR-21-review
marjev
committed
0a7d86e