Refactor `make_report` - Githubissues

opensafely-actions / cohort-report

Cohort-report generates a report for each variable in an input file

MIT License

0 stars 0 forks source link

Refactor `make_report` #37

Closed iaindillingham closed 3 years ago

iaindillingham commented 3 years ago

Let's remove the input_file_name argument to make_report, because the same information is contained within the path argument. Less code = fewer bugs.

Write tests for make_report and run_action, which is the only place where make_report is called
Remove the input_file_name argument
Tidy up the typings
Tidy up the docstring