Remove conda from ci-tests

[christopher-wild]

No description provided.

[github-actions]

🦙 MegaLinter status: ⚠️ WARNING

Descriptor	Linter	Files	Fixed	Errors	Elapsed time
⚠️ PYTHON	black	29		1	1.31s
✅ PYTHON	pylint	29		0	4.58s

See detailed report in MegaLinter reports

MegaLinter is graciously provided by

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.60%. Comparing base (d74e853) to head (837b1ce).
Report is 1 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #283      +/-   ##
==========================================
- Coverage   95.60%   95.60%   -0.01%     
==========================================
  Files          22       22              
  Lines        1525     1524       -1     
==========================================
- Hits         1458     1457       -1     
  Misses         67       67              

see 1 file with indirect coverage changes

---

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4a86d9c...837b1ce. Read the comment docs.

[christopher-wild]

Chosen to remove Conda rather than fix it. I think our GH Action should use as many inbuilt features as possible and not introduce dependencies.

[f-allian]

See comments below

[f-allian]

@christopher-wild This isn't doing what it should be doing. Have a look at the output logs for e.g. Python 3.10 on Mac OS. Under install dependencies / test with PyTest, it's running the wrong version of python (3.12 in this case even though 3.10 is requested from actions/setup-python@v5.

Their docs say:

The python-version input is optional. If not supplied, the action will try to resolve the version from the default .python-version file. If the .python-version file doesn't exist Python or PyPy version from the PATH will be used. The default version of Python or PyPy in PATH varies between runners and can be changed unexpectedly so we recommend always setting Python version explicitly using the python-version or python-version-file inputs.

So it might be a simple fix by just changing python-version: ${{ matrix.python }} to python-version: ${{ matrix.python-version }}

[f-allian]

Nice!