Mostly just for fun. I don't particularly like the UI for travis.
Added a bit extra.
1) More extensive linting.
Black formatting now enforced for python code
cargo clippy run on rust code
cargo fmt run on rust code
2) Testing now done on mac and windows
pyn5 python module was moved into seperate "python" directory. i.e. instead of "pyn5/datasets.py" it is "python/pyn5/datasets.py".
This was done for testing purposes. Previously running pip install . would build and install the package, but running pytest tests would use the local version. This would fail if maturin develop had not yet been called since the rust code would not yet be exposed. We were just testing using the local version which was fine, but maturin doesn't allow maturin develop unless called from within a virtual environment. Testing the installed version also seemed like better practice as that should be the default use case.
switch from travis to github actions.
Mostly just for fun. I don't particularly like the UI for travis. Added a bit extra. 1) More extensive linting.
pyn5 python module was moved into seperate "python" directory. i.e. instead of "pyn5/datasets.py" it is "python/pyn5/datasets.py". This was done for testing purposes. Previously running
pip install .would build and install the package, but runningpytest testswould use the local version. This would fail ifmaturin develophad not yet been called since the rust code would not yet be exposed. We were just testing using the local version which was fine, but maturin doesn't allowmaturin developunless called from within a virtual environment. Testing the installed version also seemed like better practice as that should be the default use case.