materialsproject / maggma

MongoDB aggregation machine
https://materialsproject.github.io/maggma/
Other
38 stars 32 forks source link

move API to optional dependency group; move OpenData to default installation #970

Closed rkingsbury closed 5 months ago

rkingsbury commented 5 months ago

WIP changes to dependency setup; checking impact on CI tests. See #928 .

codecov[bot] commented 5 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 88.77%. Comparing base (57fb676) to head (5bb459c). Report is 1 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #970 +/- ## ======================================= Coverage 88.77% 88.77% ======================================= Files 46 46 Lines 3981 3981 ======================================= Hits 3534 3534 Misses 447 447 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

rkingsbury commented 5 months ago

Sorry all, I did not intend to merge this yet. (Perils of using GitHub on my phone!) I'll revert if @munrojm objects to these changes. It hasn't been released so shouldn't affect any users yet.