search

SAP / odfuzz

Apache License 2.0
18 stars 12 forks source link

use environment variables, no config.yaml #35

Closed phanak-sap closed 4 years ago

phanak-sap commented 4 years ago

Ditching config.yaml in favor for environment variables to configure odfuzz runtime.

See issue #25 for details:

codecov-io commented 4 years ago

Codecov Report

Merging #35 into master will increase coverage by 2.76%. The diff coverage is 56%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #35      +/-   ##
==========================================
+ Coverage   31.63%   34.39%   +2.76%     
==========================================
  Files          15       15              
  Lines        3142     3259     +117     
==========================================
+ Hits          994     1121     +127     
+ Misses       2148     2138      -10
Impacted Files Coverage Δ
odfuzz/fuzzer.py 26.88% <0%> (+2.56%) :arrow_up:
odfuzz/arguments.py 100% <100%> (ø) :arrow_up:
odfuzz/constants.py 100% <100%> (ø) :arrow_up:
odfuzz/config.py 60% <25%> (+4%) :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 2844593...11125ff. Read the comment docs.

phanak-sap commented 4 years ago

reviewed offline with @jzeinert-sap