Skip to content
This repository has been archived by the owner on Dec 20, 2024. It is now read-only.

MAINT: Do not run unit tests in all platforms #240

Merged
merged 1 commit into from
Oct 23, 2024
Merged

Conversation

oesteban
Copy link
Member

Revert the test matrix of #234 for two reasons:

  • It doesn't make a lot of sense to run all unittests for different Python versions when most of our problems will come from elsewhere as regards the platform (ANTs, data properly installed)
  • We don't have a critical mass of functional code working, so tests fail for spurious installation reasons and we spend time focusing on that as opposed to getting eddymotion to a functional state.

For the time being, let's make sure it works with 3.12 and reconsider when the focus is portability.

Copy link

codecov bot commented Oct 23, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 64.08%. Comparing base (24ce39e) to head (ffd8067).
Report is 2 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #240   +/-   ##
=======================================
  Coverage   64.08%   64.08%           
=======================================
  Files          18       18           
  Lines         905      905           
  Branches      117      117           
=======================================
  Hits          580      580           
  Misses        282      282           
  Partials       43       43           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@oesteban oesteban merged commit 04f0941 into main Oct 23, 2024
9 checks passed
@oesteban oesteban deleted the fix/tox-revert branch October 23, 2024 10:49
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant