Skip to content

docs: fix MkDocs protobuf reference for ConstantLayout#5833

Merged
Xuanwo merged 1 commit intomainfrom
fix/mkdocs-protobuf-constant-layout
Jan 28, 2026
Merged

docs: fix MkDocs protobuf reference for ConstantLayout#5833
Xuanwo merged 1 commit intomainfrom
fix/mkdocs-protobuf-constant-layout

Conversation

@Xuanwo
Copy link
Copy Markdown
Collaborator

@Xuanwo Xuanwo commented Jan 28, 2026

Fixes the docs build failure in CI caused by an outdated protobuf template reference.


Parts of this PR were drafted with assistance from Codex (with gpt-5.2) and fully reviewed and edited by me. I take full responsibility for all changes.

@github-actions github-actions Bot added the documentation Improvements or additions to documentation label Jan 28, 2026
@github-actions
Copy link
Copy Markdown
Contributor

PR Review Summary

Status: LGTM

This is a straightforward documentation fix that updates a stale protobuf template reference from AllNullLayout to ConstantLayout.

Verified:

  • The ConstantLayout message exists in protos/encodings_v2_1.proto:156
  • The old AllNullLayout message no longer exists (only referenced in a comment explaining the evolution)
  • The documentation text updates correctly describe the generalized behavior (constant values, not just all-null)

No concerns with this change.

@Xuanwo Xuanwo merged commit b0396c3 into main Jan 28, 2026
8 checks passed
@Xuanwo Xuanwo deleted the fix/mkdocs-protobuf-constant-layout branch January 28, 2026 10:40
vivek-bharathan pushed a commit to vivek-bharathan/lance that referenced this pull request Feb 2, 2026
…5833)

Fixes the docs build failure in CI caused by an outdated protobuf
template reference.

---

**Parts of this PR were drafted with assistance from Codex (with
`gpt-5.2`) and fully reviewed and edited by me. I take full
responsibility for all changes.**
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants