Skip to content

Update classes on protobufs update#4805

Merged
caveman99 merged 2 commits into
meshtastic:masterfrom
scruplelesswizard:update-classes-on-protobuf-update
Sep 21, 2024
Merged

Update classes on protobufs update#4805
caveman99 merged 2 commits into
meshtastic:masterfrom
scruplelesswizard:update-classes-on-protobuf-update

Conversation

@scruplelesswizard

Copy link
Copy Markdown
Contributor

This PR changes the update process for Protobufs. Every time a change is made to the meshtastic submodule in a PR, the regen-protos.sh will be run and the changes committed to the PR's branch.

This enables class regeneration on Dependabot's PRs and also on any other users PR where meshtastic/ is updated. Additionally it ensures that on any PRs where the protobufs have changed that the classes match.

@scruplelesswizard scruplelesswizard force-pushed the update-classes-on-protobuf-update branch from e8a7197 to 2026992 Compare September 21, 2024 14:51
Comment thread .github/workflows/update_protobufs.yml Outdated
Comment thread .github/workflows/update_protobufs.yml Outdated
@caveman99

caveman99 commented Sep 21, 2024

Copy link
Copy Markdown
Member

IS this correct? the complete submodule path is /protobufs/meshtastic/**

The other submodule is meshtestic, the test suite...

@scruplelesswizard

Copy link
Copy Markdown
Contributor Author

Oh, totally not. I'll get a patch up shortly! Thanks for checking @caveman99

@scruplelesswizard

Copy link
Copy Markdown
Contributor Author

PR closed unintentionally

@scruplelesswizard scruplelesswizard force-pushed the update-classes-on-protobuf-update branch from ea34add to b9e9380 Compare September 21, 2024 15:50
@scruplelesswizard scruplelesswizard force-pushed the update-classes-on-protobuf-update branch from b9e9380 to 2072ebd Compare September 21, 2024 16:58
@caveman99 caveman99 merged commit 01203eb into meshtastic:master Sep 21, 2024
@scruplelesswizard scruplelesswizard deleted the update-classes-on-protobuf-update branch September 28, 2024 19:10
jeek pushed a commit to jeek/Meshtastic-Exploiteers-Hacker-Pager that referenced this pull request Jun 30, 2026
…ses-on-protobuf-update

Update classes on protobufs update
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants