search

adafruit / Wippersnapper_Protobuf

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

Don't submit wrapper pull requests to WS Repos for every PR on this repo #122

Open brentru opened 1 year ago

brentru commented 1 year ago

This (https://github.com/adafruit/Wippersnapper_Protobuf/pull/121#pullrequestreview-1192348381) should not have submitted a pull request. We may want to update the workflow to only submit wrapper pull requests on release, and create a new release for it.

Not sure how to parse it out otherwise, suppose we could check if the PR contains .proto files.

@lorennorman - thoughts?

lorennorman commented 1 year ago

Yep we can check if .proto files were modified. I do similar stuff in the components and boards repos to make sure only certain files are modified, it starts with the gather-relevant-files job and the files list gets input into later steps. So we have a jumping off point when we're ready to fix this.