Fix CI warnings #147

lilyminium · 2024-10-17T05:30:55Z

Fixes #148

CI is getting warnings about using macOS-latest and Mambaforge; this PR upgrades them both. It also removes some Py3.12 exclusions that I think are ready to go now.
It also switches over to use conda-forge/dgl which is a fully featured install that includes torchdata. Previously that was causing an ImportError on torchdata, which triggered the pytest.importorskip(dgl) condition of some tests.

PR Checklist

codecov-commenter · 2024-10-17T05:34:37Z

All modified and coverable lines are covered by tests ✅

Project coverage is 64.14%. Comparing base (1d7e4ff) to head (1cbe6bd).
Report is 2 commits behind head on main.

Additional details and impacted files

lilyminium added 2 commits October 17, 2024 16:30

update ci file

e09bbcf

specify mamba version?

f2463c8

lilyminium added 4 commits October 17, 2024 16:42

add channel priority

721b2af

switch to micromamba for conda test

4b010bd

remove failing conda call

4f1cc29

update everything to micromamba

d811159

lilyminium mentioned this pull request Oct 18, 2024

Pytest not collecting all tests #148

Closed

lilyminium added 4 commits October 18, 2024 11:26

get rid of xdist for now

f1b524f

does adding init.py make a difference?

aae87c9

add step checking dgl install

13b29a4

switch to conda-forge dgl

1cbe6bd

lilyminium merged commit 4347b29 into main Oct 18, 2024
62 of 116 checks passed

lilyminium deleted the update-ci branch October 18, 2024 07:15