adafruit / Wippersnapper_Protobuf

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

Adds reuseable workflow #116

Closed brentru closed 1 year ago

brentru commented 1 year ago

Solves the first bullet point from issue https://github.com/adafruit/Wippersnapper_Protobuf/issues/105 by creating a reusable workflow and having the build action depend upon the backward compatibility action passing.

buf-checks.yml should be run prior to protoc-wrapper-generation.yml. Currently, they run independently of each other, letting a file without backward-compatibility get generated and deployed to multiple repositories. This is still useful when working on new message types, but harmful when updating existing ones.

@lorennorman We may want to wait on adding this to the repo because requiring backward-compat. first, while we're still in beta (and breaking things) may cause release issues/delays. Could also modify on:, not sure how you'd like to proceed.