search

mikepurvis / catkin_tools_document

Experimental parallel documentation plugin for catkin_tools
Apache License 2.0
8 stars 5 forks source link

Simplify loop of packages #21

Closed MatthijsBurgh closed 2 years ago

MatthijsBurgh commented 3 years ago

packages_to_be_documented is already in topological order so we can just loop over it. Instead of looping all packages and checking if it is in packages_to_be_documented

MatthijsBurgh commented 3 years ago

@mikepurvis have you been able to take a look at this?

MatthijsBurgh commented 2 years ago

@mikepurvis friendly ping ;)

MatthijsBurgh commented 2 years ago

@mikepurvis friendly ping ;)

mikepurvis commented 2 years ago

Looks reasonable enough.