jmscslgroup / bagpy

Python package for reading, and extracting data from rosbag files and performing any analysis on it.
https://jmscslgroup.github.io/bagpy/
Other
207 stars 39 forks source link

Move documentation dependencies to extras_require, add testing #31

Closed betaBison closed 2 years ago

betaBison commented 2 years ago

Thanks @rahulbhadani for maintaining bagpy over the years.

I'd like to add a dependency on bagpy in one of my projects, but the dependency on a pinned Sphinx version is incompatible with my other project dependencies.

This pull request:

Note: I didn't know how to test the .Travis.yml before creating a pull request, so please let me know how to verify the install update I made to it.

betaBison commented 2 years ago

fixes #30

betaBison commented 2 years ago

I tried (and failed) to fix Travis CI. You can see the documentation builds passing with the new build-docs workflow here: https://github.com/betaBison/bagpy/runs/7925454189?check_suite_focus=true

betaBison commented 2 years ago

Anything I can do to push this pull request through @rahulbhadani ?