At the moment tests are run on PRs and on commits to develop. This results in multiple bot comments with test coverage, we should change this so that we have. We are also updating the tests passing badge for every action run, instead of just for the master branch.
I think we need 3 similar actions workflows:
Pull request tests - runs pytest on PR, reports coverage to PR, does not create any badges
Something to check develop is passing after PR merges - runs pytest on commit to develop, does not report coverage or create coverage badge, does create 'develop tests passing' badge (this will need to be added to readme)
Master tests and coverage - runs on commit to master, creates badge for master tests passing and creates badge for coverage - both reported on readme.
At the moment tests are run on PRs and on commits to develop. This results in multiple bot comments with test coverage, we should change this so that we have. We are also updating the tests passing badge for every action run, instead of just for the master branch.
I think we need 3 similar actions workflows: