Skip to content

feat: make SnapshotUtil a public installed header - #4

Open
smaheshwar-pltr wants to merge 1 commit into
mainfrom
make-snapshot-util-public
Open

smaheshwar-pltr wants to merge 1 commit into
mainfrom
make-snapshot-util-public

Conversation

@smaheshwar-pltr

@smaheshwar-pltr smaheshwar-pltr commented Apr 16, 2026 •

Copy link
Copy Markdown
Owner

What

Rename snapshot_util_internal.h to snapshot_util.h, update its callers, and add it to the Meson install list.

Why

SnapshotUtil is exported public API, but the old filename caused the installation rules to exclude its header. Installed-package consumers need these snapshot ancestry, time-travel, and schema lookup helpers.

Closes #8.

Validation

The full GitHub Actions matrix passed.

AI assistance: Codex was used to rebase and update the existing change for current main. I reviewed the resulting diff and validation.

@smaheshwar-pltr
smaheshwar-pltr force-pushed the make-snapshot-util-public branch from 7ffc6d8 to 4908911 Compare April 16, 2026 22:47
@smaheshwar-pltr
smaheshwar-pltr marked this pull request as draft April 16, 2026 22:48
@smaheshwar-pltr
smaheshwar-pltr marked this pull request as ready for review April 17, 2026 01:31
@smaheshwar-pltr smaheshwar-pltr mentioned this pull request May 30, 2026
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.

SnapshotUtil header not installed — filename contains 'internal'

1 participant