Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix GH Actions tests #652

Merged
merged 7 commits into from Apr 9, 2024
Merged

Fix GH Actions tests #652

merged 7 commits into from Apr 9, 2024

Conversation

ethanwhite
Copy link
Member

This fixes our failing GH Actions tests:

  1. Installs required system files (something in the default build environment must have changed) (Fixes Check if version limit on setuptools can be removed from environment.yml #631)
  2. Updates setup-micromamba usage, which changed upstream
  3. Install pycocotools from pip since conda install is failing on Python 3.12
  4. Uses separate runs for tests, style, and notebook checks so that failing tests actually result in a failed build (Fixes GH Actions build passing when tests erroring #651)
  5. Fixes some yapf style issues and a missing dependency to get the test passing

OpenCV uses system libraries on Linux
Both argument names and ability to use version 3.x changed upstream
Installing from conda-forge was erroring on Python 3.12
The change in 2cbb072 was valid so this reimplements it.
It just looked like this change was causing an error due to an upstream change in the testing setup.
Running them together was causing the build to pass even when tests were failing.

Fixes weecology#651.
@bw4sz bw4sz merged commit 6b1c199 into weecology:main Apr 9, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants