tethysplatform / tethys

The Tethys Platform main Django website project repository.
http://tethysplatform.org/
BSD 2-Clause "Simplified" License
92 stars 49 forks source link

Optional Dependencies #967

Closed swainn closed 9 months ago

swainn commented 1 year ago

This PR makes most Python dependencies optional.

TODO:

coveralls commented 1 year ago

Coverage Status

coverage: 100.0%. remained the same when pulling 98b0ca37ae6544cc7acb9d847fc308b58b5b87d3 on optional_dependencies into f62cebb3d6961ffd34d59bdebf938bd2d3a097a8 on main.

swainn commented 1 year ago

Add instructions to the documentation for installing dependencies for the following features:

swainn commented 1 year ago

Tutorial language: "...this package may not be included in some Tethys installations, so you should include it as a dependency of your app to ensure it is installed."

sdc50 commented 11 months ago

@swainn We need to rebase this branch on master. I'm inclined to squash the commits and then rebase and force push. That would require you (and anyone else that has this branch checked out) to re-set it (or delete and re-checkout). Do you have any changes to push before I do that?

swainn commented 11 months ago

@swainn We need to rebase this branch on master. I'm inclined to squash the commits and then rebase and force push. That would require you (and anyone else that has this branch checked out) to re-set it (or delete and re-checkout). Do you have any changes to push before I do that?

Sounds like a good plan to me. I don't have any changes that I need to merge.

sdc50 commented 9 months ago

fixes #980 and fixes #985

sdc50 commented 9 months ago

@swainn I've finished the docs for this PR, so it should be ready to go.