Skip to content

Clean up coverage writing#84

Open
jspaaks wants to merge 5 commits intomainfrom
clean-up-coverage-writing
Open

Clean up coverage writing#84
jspaaks wants to merge 5 commits intomainfrom
clean-up-coverage-writing

Conversation

@jspaaks
Copy link
Copy Markdown
Member

@jspaaks jspaaks commented Mar 11, 2021

coverage was written into various places, this PR cleans that up. Everything is now under /coverage, like so:

coverage/
├── .coverage
├── coverage.xml
├── htmlcov
│   ├── coverage_html.js
│   ├── fairtally_check_py.html
│   ├── fairtally_cli_py.html
│   ├── fairtally___init___py.html
│   ├── fairtally_redirect_stdout_stderr_py.html
│   ├── fairtally_utils_py.html
│   ├── fairtally___version___py.html
│   ├── favicon_32.png
│   ├── index.html
│   ├── jquery.ba-throttle-debounce.min.js
│   ├── jquery.hotkeys.js
│   ├── jquery.isonscreen.js
│   ├── jquery.min.js
│   ├── jquery.tablesorter.min.js
│   ├── keybd_closed.png
│   ├── keybd_open.png
│   ├── status.json
│   └── style.css
└── README.md

The configuration is now more explicit, in /.coveragerc

@jspaaks jspaaks marked this pull request as ready for review March 11, 2021 08:52
Co-authored-by: Stefan Verhoeven <s.verhoeven@esciencecenter.nl>
@jspaaks
Copy link
Copy Markdown
Member Author

jspaaks commented Mar 16, 2021

Suggestions committed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants