Skip to content
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

Validate notebooks in CI #74

Merged
merged 4 commits into from
Feb 28, 2019
Merged

Conversation

mdekstrand
Copy link
Member

@mdekstrand mdekstrand commented Feb 27, 2019

This is to help ensure #66 doesn't recur.

@codecov-io
Copy link

codecov-io commented Feb 27, 2019

Codecov Report

Merging #74 into master will increase coverage by 0.14%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #74      +/-   ##
==========================================
+ Coverage   95.97%   96.12%   +0.14%     
==========================================
  Files          21       21              
  Lines        2039     2039              
==========================================
+ Hits         1957     1960       +3     
+ Misses         82       79       -3
Impacted Files Coverage Δ
lenskit/batch/_predict.py 100% <0%> (+6.97%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0642bbe...ecea752. Read the comment docs.

mdekstrand and others added 2 commits February 28, 2019 09:14
- Set PYTHONPATH so doc validation can find LensKit
- Re-run the Getting Started notebook
@codeclimate
Copy link

codeclimate bot commented Feb 28, 2019

Code Climate has analyzed commit ecea752 and detected 0 issues on this pull request.

View more on Code Climate.

@mdekstrand mdekstrand added this to the 0.6.0 milestone Feb 28, 2019
@mdekstrand mdekstrand merged commit 45e6509 into lenskit:master Feb 28, 2019
@mdekstrand mdekstrand deleted the feature/check-docs branch February 28, 2019 17:48
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.

None yet

2 participants