Skip to content

install dependencies through rosdep ci - #862

Merged
whoenig merged 5 commits into
mainfrom
check-rosdep-ci
Jun 7, 2026
Merged

install dependencies through rosdep ci#862
whoenig merged 5 commits into
mainfrom
check-rosdep-ci

Conversation

@knmcguire

@knmcguire knmcguire commented Jun 3, 2026

Copy link
Copy Markdown
Collaborator

Fixes current ci issues in the buildfarm.

  • install depencencies with rosdep in ci
  • add crazyflie_server_py to ci
  • remove ament_python from crazyflie_server_py
  • fix flake8 and other test issues in crazyflie_server_py
  • Removed dependency on tf_transformations (broke with latest numpy)

@knmcguire
knmcguire force-pushed the check-rosdep-ci branch 2 times, most recently from 90ee1c1 to f621186 Compare June 3, 2026 11:14
@knmcguire

Copy link
Copy Markdown
Collaborator Author

Almost there. Seems to be still something wrong in lyrical and rolling but will focus on that friday

@knmcguire
knmcguire force-pushed the check-rosdep-ci branch 2 times, most recently from c60203a to e992700 Compare June 5, 2026 09:38
@knmcguire
knmcguire requested a review from whoenig June 6, 2026 18:55
@knmcguire knmcguire self-assigned this Jun 6, 2026
@knmcguire
knmcguire marked this pull request as ready for review June 6, 2026 18:55
@knmcguire

Copy link
Copy Markdown
Collaborator Author

I quickly tested connection with a crazyflie with the updated flake8errorfree server. If it is all right with you, and the build/test goes well @whoenig, just go ahead and merge.

@whoenig
whoenig merged commit 99e3853 into main Jun 7, 2026
6 checks passed
@whoenig
whoenig deleted the check-rosdep-ci branch June 7, 2026 15:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants