search

SINTEF / ci-cd

Callable GitHub Actions workflows & pre-commit hooks
https://SINTEF.github.io/ci-cd
MIT License
0 stars 1 forks source link

Make `test: true` actually work for "CD - Release" #83

Closed CasperWA closed 1 year ago

CasperWA commented 1 year ago

Setting test: true is not enough for testing the CD - Release workflow, unfortunately. One must now still uncomment a lot of if statements concerning checking the github.ref value.

CasperWA commented 1 year ago

I think this is a misconception. The test variable is meant to still test a proper release, but instead release to TestPyPI and write out debug statements.

In order to test the CD - Release workflow in another way, other, more invasive, things should be done.

It's worth wondering whether it is the workflow or other parts one truly wants to test. E.g., in the current #82 we don't want to test the workflow (only perhaps that the parsing is good from the workflow description to the ci-cd tool). Instead we truly just want to run the ci-cd tool and understand whether the changes are correct. This can be achieved by installing the tool locally and running it, of course.