Skip to content

Actions: Update protobufs using the triggering branch#10612

Merged
thebentern merged 1 commit into
masterfrom
update-protos-branch
Jun 8, 2026
Merged

Actions: Update protobufs using the triggering branch#10612
thebentern merged 1 commit into
masterfrom
update-protos-branch

Conversation

@vidplace7

Copy link
Copy Markdown
Member

This pull request updates the workflow for updating protobuf submodules to better handle branch references and submodule updates. The main change is switching to use github.ref_name instead of github.ref and improving how the protobuf submodule is updated. Now when the workflow is triggered in develop branch, develop branch protobufs will be used.

Workflow improvements:

  • The condition for updating the submodule now uses github.ref_name for better clarity and reliability when checking if the workflow is running on master or develop branches.
  • The submodule update step now sets the working directory to protobufs, uses an environment variable (GIT_BRANCH) to track the triggering branch, and replaces the previous git submodule update command with a git pull and git checkout to ensure the correct branch is checked out and up to date.

@vidplace7 vidplace7 requested review from Copilot and thebentern June 2, 2026 21:40
@vidplace7 vidplace7 added github_actions Pull requests that update GitHub Actions code bugfix Pull request that fixes bugs labels Jun 2, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the update_protobufs.yml GitHub Actions workflow to select the protobufs submodule branch based on the triggering workflow branch, aiming to ensure develop runs use develop protobufs (and likewise for master).

Changes:

  • Switch workflow branch checks from github.ref to github.ref_name for clearer branch matching.
  • Update the protobufs submodule update step to run within protobufs/ and attempt to pull/checkout the triggering branch.

Comment thread .github/workflows/update_protobufs.yml Outdated
@vidplace7 vidplace7 force-pushed the update-protos-branch branch from d435161 to 7bf1530 Compare June 5, 2026 18:09
@thebentern thebentern merged commit 5c76902 into master Jun 8, 2026
84 checks passed
@thebentern thebentern deleted the update-protos-branch branch June 8, 2026 19:53
Evil8it pushed a commit to Evil8it/ME4TACTNK that referenced this pull request Jun 10, 2026
raghumad pushed a commit to raghumad/mezulla-firmware that referenced this pull request Jun 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix Pull request that fixes bugs github_actions Pull requests that update GitHub Actions code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants