search

adafruit / Wippersnapper_Protobuf

Protocol buffer messages for Adafruit.io WipperSnapper Beta
MIT License
0 stars 2 forks source link

Flatten long filepaths #138

Closed brentru closed 9 months ago

brentru commented 9 months ago

Protobuf mimics the way Java structures packages and naming, but it causes very long identifiers in the code. This leads to long identifiers being called, which is bad for readability and overall code size.

This pull request aims to flatten the proto/wippersnapper/protoName/v1/protoName.proto directory into proto/wippersnapper/*

brentru commented 9 months ago

Good catch on the plural in options file for sensor events. Presumably protolint doesn't validate not found entries in .options files

Correct, runningprotoc to generate files was what found them.