search

foliant-docs / foliant

Comprehensive markdown-based documentation toolkit
https://foliant-docs.github.io/docs/
MIT License
160 stars 9 forks source link

add loader to all yaml.load calls #47

Closed Vanderhoof closed 5 years ago

Vanderhoof commented 5 years ago

In old PyYAML versions function yaml.load had default value for loader parameter equalilng yaml.Loader. In new PyYAML 5.1 using yaml.load without specifying the loader parameter explicitly is not allowed anymore. In this pull request I added loader=yaml.Loader param to all yaml.load calls so now user can safely upgrade to PyYAML 5.1

codecov[bot] commented 5 years ago

Codecov Report

Merging #47 into develop will not change coverage. The diff coverage is 60%.

Impacted file tree graph

@@           Coverage Diff            @@
##           develop      #47   +/-   ##
========================================
  Coverage    75.42%   75.42%           
========================================
  Files           13       13           
  Lines          350      350           
========================================
  Hits           264      264           
  Misses          86       86
Impacted Files Coverage Δ
foliant/config/base.py 100% <100%> (ø) :arrow_up:
foliant/config/include.py 44.44% <33.33%> (ø) :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 e289984...f063228. Read the comment docs.