Code review comment for lp://staging/~thomas-voss/location-service/add-ubx-provider

Revision history for this message
Scott Sweeny (ssweeny) wrote :

Overall the code LGTM.

Could you clarify why you removed the LGPL test and explain a bit about the new commands?

Thanks.

review: Needs Information (code)

« Back to merge proposal