Ryan
Ryan
Here's a test plan based on the changes, I'll check off things as I go. I might request help on some of the modules like ntrip that I don't know...
> @peterbarker @Ryanf55 Your reviews, please. > > % `pre-commit run --all-files` should pass on this branch! Sorry, it does not pass. Any tips? I've never seen this until the...
> pre-commit run --all-files flake8 I tried upgrading pre-commit, it still doesn't work. ``` ryan@B650-970:~/Dev/ardu_ws/src/ardupilot$ pre-commit run --all-files flake8 [INFO] Initializing environment for https://github.com/pycqa/flake8. [INFO] Installing environment for https://github.com/pycqa/flake8. [INFO]...
> python -m pre-commit run --all-files ``` $ python --version Python 3.10.12 $ python -m pre-commit --version /usr/bin/python: No module named pre-commit which pre-commit /home/ryan/.local/bin/pre-commit ```
My pre-commit environment is hosed, feel free to merge based on other approvals.
> > My pre-commit environment is hosed, feel free to merge based on other approvals. > > This does not fill me with confidence. tridge also ended up with a...
I ran into this releasing grid_map_demos
@maximilianwulf Can you please merge?
> @maximilianwulf Can you please merge? Friendly ping @maximilianwulf - this is a quick README change ready for merge.
@wep21 Looks like nav2 broke ABI again with their rosdep sources. Thoughts on only compile nav2 from source on rolling?