search

hastexo / olx-utils

Tools facilitating the creation of Open edX courseware in the Open Learning XML (OLX) format
GNU Affero General Public License v3.0
5 stars 5 forks source link

Improve tasks #39

Closed fghaas closed 5 years ago

fghaas commented 5 years ago

These are just a couple of commits I've had sitting around for a little while, but never sent a PR. They slightly improve the handling of environment variables and logging, when being called not from the CLI, but from an invoke task.

codecov-io commented 5 years ago

Codecov Report

Merging #39 into master will increase coverage by <.01%. The diff coverage is 76.92%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master     #39      +/-   ##
=========================================
+ Coverage   97.89%   97.9%   +<.01%     
=========================================
  Files          15      15              
  Lines         854     858       +4     
=========================================
+ Hits          836     840       +4     
  Misses         18      18
Impacted Files Coverage Δ
olxutils/cli.py 94.55% <76.92%> (+0.15%) :arrow_up:

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 2f3f215...42c71b8. Read the comment docs.

fghaas commented 5 years ago

@arbrandes Cool; when you merge this, would you mind cutting a 0.3.1 release?

fghaas commented 5 years ago

Thank you!