issues
search
mincong-h
/
finance-toolkit
Finance Toolkit
3
stars
1
forks
source link
Code clean up
#13
Closed
mincong-h
closed
4 years ago
mincong-h
commented
4 years ago
... last PR of this weekend 😬
Move test resources (CSV files) to "download" directory
Reduce usage of fixture "location" and replace by cfg instead
Remove
dest_dir
from params of Pipeline.run() because it's already provided by
cfg
Use Configuration to initialize Summary
Extract common code in pipeline and avoid duplicated code
... last PR of this weekend 😬
dest_dirfrom params of Pipeline.run() because it's already provided bycfg