[N/A] Wrote test for feature
[N/A] Added changes to CHANGELOG.md
[N/A] Bumped version number (delete if unneeded)
Changes proposed:
We previously only ran the test on pushes to the repo (including non-master branch pushes). However, as PRs come in, they are usually pushed to a forked repo and then made a PR from that forked branch - thereby avoiding the PR push. This would allow us to run the automated test suite on any PR created as well.
Coverage: 91.76%. Remained the same when pulling 1d1862951ef3c35f9bab86d9c9016493d2e38fc4 on adnanhemani:ahemani/update_tests_to_run_on_pr into 60385d1665d541b8bb64e6ab68f21044268d5a4d on data-8:master.
[N/A] Wrote test for feature [N/A] Added changes to CHANGELOG.md [N/A] Bumped version number (delete if unneeded)
Changes proposed: We previously only ran the test on pushes to the repo (including non-master branch pushes). However, as PRs come in, they are usually pushed to a forked repo and then made a PR from that forked branch - thereby avoiding the PR push. This would allow us to run the automated test suite on any PR created as well.