Skip to content

problem: not building on CI - #3

Merged
splix merged 1 commit into
masterfrom
ci/initial
May 5, 2026
Merged

problem: not building on CI#3
splix merged 1 commit into
masterfrom
ci/initial

Conversation

@splix

@splix splix commented May 5, 2026

Copy link
Copy Markdown
Member

No description provided.

Copilot AI review requested due to automatic review settings May 5, 2026 21:32
@splix
splix merged commit 83312e5 into master May 5, 2026
3 checks passed
@splix
splix deleted the ci/initial branch May 5, 2026 21:32

Copilot AI left a comment

Copy link
Copy Markdown

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 GitHub Actions CI workflows to ensure builds/publishes succeed when the repository depends on a checked-out git submodule (and, for publishing, required system tooling).

Changes:

  • Enable actions/checkout submodule checkout in both test and publish workflows.
  • Add protobuf compiler installation to the publish workflow to support builds that require protoc.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
.github/workflows/test.yaml Checks out git submodules during CI test runs so builds/tests can access submodule content.
.github/workflows/publish.yaml Checks out git submodules and installs protobuf-compiler before cargo publish.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

run: rustup toolchain add --profile=minimal stable

- name: Install system libs
run: sudo apt-get install protobuf-compiler
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.

2 participants