ci: clean old comments

Fixes #181 (closed)

@katsanto or @gypas or @kanitz : please take a look

changed milestone to %pipeline

added Discuss Doing Testing & CI/CD labels

changed the description

@bakma shouldn't you also remove the respective scripts? from the tests folder?

Agree with @katsanto. Also wondering about the ALFA tests. Guess I missed the discussion where it was decided to skip these. Are these integrated into other tests now? Does @burri0000 agree with removing?

The respective discussion is in #181 (closed)

Thanks - good for me then, after removing the unused scripts

@bakma If you remove the lab-key test, don't forget to remove https://git.scicore.unibas.ch/zavolan_group/pipelines/zarp/-/blob/dev/install/environment.dev.yml#L13

added 1 commit

• 32c933a3 - refactor: remove everything labkey-related

Compare with previous version

added 1 commit

• 5f511263 - refactor: remove old ALFA unit test

Compare with previous version

Right, I did not pay much attention previously, I just adjusted the CI ¯\(ツ)/¯

I have pushed a new version which removes everything related to the now-unused tests (scripts, test dirs, docs).

Please take a look and Approve if OK, if not let me know and we iterate once again.

approved this merge request

merged

mentioned in commit 4300f2fc

mentioned in issue #181 (closed)

mentioned in merge request !100 (merged)