Skip to content

fix(ci): fix ci by install protobuf-compiler#2

Merged
jja725 merged 2 commits intolance-format:mainfrom
jja725:feat/initial-port
Mar 25, 2026
Merged

fix(ci): fix ci by install protobuf-compiler#2
jja725 merged 2 commits intolance-format:mainfrom
jja725:feat/initial-port

Conversation

@jja725
Copy link
Copy Markdown
Collaborator

@jja725 jja725 commented Mar 24, 2026

Summary

  • Install protobuf-compiler in all Linux CI jobs (clippy, test, rustdoc, MSRV)
  • Install protobuf via brew in the macOS CI job

Motivation

lance-encoding has a build script that requires protoc. Without it, CI fails with:

Could not find `protoc`. If `protoc` is installed, try setting the `PROTOC` environment variable...

Test plan

  • CI should pass after this change

🤖 Generated with Claude Code

@jja725 jja725 force-pushed the feat/initial-port branch from df9f87b to 27d3491 Compare March 24, 2026 19:59
@jja725 jja725 changed the title feat: port lance-c crate from lance monorepo fix(ci): install protobuf-compiler for lance-encoding build Mar 24, 2026
@jja725 jja725 force-pushed the feat/initial-port branch from 27d3491 to ba92b41 Compare March 24, 2026 20:06
lance-encoding's build script requires protoc. Install it via
apt-get on Linux and brew on macOS.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@jja725 jja725 force-pushed the feat/initial-port branch 3 times, most recently from d6e49ea to 31140e1 Compare March 24, 2026 20:15
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@jja725 jja725 force-pushed the feat/initial-port branch from 31140e1 to 65205c6 Compare March 24, 2026 20:17
@jja725
Copy link
Copy Markdown
Collaborator Author

jja725 commented Mar 24, 2026

@Xuanwo do you mind take a look when you are free? Thanks

@jja725 jja725 changed the title fix(ci): install protobuf-compiler for lance-encoding build fix(ci): fix ci by install protobuf-compiler Mar 24, 2026
@jja725 jja725 merged commit 8e7811b into lance-format:main Mar 25, 2026
7 checks passed
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.

1 participant