helpers icon indicating copy to clipboard operation
helpers copied to clipboard

Review / improve the logic for skipping tests

Open gpsaggese opened this issue 9 months ago • 2 comments

Right now the logic for running tests specific of a repo / machine is pretty messy.

We want to:

  • [ ] Explain how it works right now (in the internal doc and maybe a note in the white paper)
  • [ ] Decide how it should work (updating the docs)
  • [ ] Clean up the code

Let's have @heanhsok @samarth9008 (and @sonniki) keep an eye on it

gpsaggese avatar Mar 23 '25 13:03 gpsaggese

We have some documentation at: - https://github.com/causify-ai/helpers/blob/master/docs/coding/all.run_unit_tests.how_to_guide.md#running-tests-on-gh-actions - And we can skip tests on submodule using invoke

@gpsaggese What is our goal here? - Do we want the logic to skip specific tests in a PR? - For which we either comment out the code or skip using pytest.

Shaunak01 avatar Mar 31 '25 16:03 Shaunak01

Let's pause this one as low priority.

gpsaggese avatar Mar 31 '25 23:03 gpsaggese