search

pylti / lti

Learning Tools Interoperability for Python
Other
78 stars 45 forks source link

Code to allow registration of an LTI 2 proxy #42

Closed subssn21 closed 7 years ago

codecov[bot] commented 7 years ago

Codecov Report

Merging #42 into master will increase coverage by 0.65%. The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #42      +/-   ##
==========================================
+ Coverage   82.34%   82.99%   +0.65%     
==========================================
  Files          15       16       +1     
  Lines         623      647      +24     
  Branches      111      113       +2     
==========================================
+ Hits          513      537      +24     
  Misses         82       82              
  Partials       28       28
Impacted Files Coverage Δ
src/lti/__init__.py 100% <100%> (ø) :arrow_up:
src/lti/launch_params.py 90% <100%> (+0.14%) :arrow_up:
src/lti/tool_proxy.py 100% <100%> (ø)

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 2762561...f7c98ee. Read the comment docs.

ryanhiebert commented 7 years ago

@subssn21 : If you accept the invitation to join the pylti organization, you should be able to merge this pull request.

ryanhiebert commented 7 years ago

@subssn21 : are you willing to join the organization and become a maintainer?

subssn21 commented 7 years ago

Sure, I would be willing to help maintain this project

ryanhiebert commented 7 years ago

Cool. I've already approved this PR, so you can perform a squash merge at your convenience.