Skip to content

trees/tiles: Add TileReader and ReadEntries - #8956

Merged
jsha merged 2 commits into
mainfrom
paving-a-path-to-a-proper-publisher-05
Aug 20, 2026
Merged

trees/tiles: Add TileReader and ReadEntries#8956
jsha merged 2 commits into
mainfrom
paving-a-path-to-a-proper-publisher-05

Conversation

@beautifulentropy

@beautifulentropy beautifulentropy commented Aug 18, 2026

Copy link
Copy Markdown
Member

Also, narrow the interface for the s3c used on read-only trees/tiles functions.

Closes #8946

@beautifulentropy
beautifulentropy marked this pull request as ready for review August 18, 2026 15:44
@beautifulentropy
beautifulentropy requested a review from a team as a code owner August 18, 2026 15:44
@beautifulentropy
beautifulentropy requested review from aarongable, ezekiel and jsha and removed request for jsha August 18, 2026 15:44
@beautifulentropy
beautifulentropy force-pushed the paving-a-path-to-a-proper-publisher-05 branch from e5dd924 to 115aebe Compare August 18, 2026 21:07
@beautifulentropy

Copy link
Copy Markdown
Member Author

Apologies for the force push. I am cherry-picking some interface changes that really belong in this PR from my working branch.

Comment thread trees/tiles/tiles_test.go
@beautifulentropy
beautifulentropy force-pushed the paving-a-path-to-a-proper-publisher-05 branch from 115aebe to 1a890cb Compare August 18, 2026 21:19
Comment thread trees/tiles/tiles.go
Comment thread trees/tiles/tiles.go
Comment thread trees/tiles/tiles.go Outdated
Comment thread trees/tiles/tiles.go Outdated
@beautifulentropy
beautifulentropy force-pushed the paving-a-path-to-a-proper-publisher-05 branch from 2882a10 to 94e5180 Compare August 19, 2026 21:18
@beautifulentropy
beautifulentropy force-pushed the paving-a-path-to-a-proper-publisher-05 branch from 94e5180 to 2798b11 Compare August 19, 2026 21:21

@aarongable aarongable 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.

Awesome, love the new EntriesForPackage.

@jsha jsha 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.

Looks great!

@jsha
jsha merged commit a6ef070 into main Aug 20, 2026
32 of 34 checks passed
@jsha
jsha deleted the paving-a-path-to-a-proper-publisher-05 branch August 20, 2026 16:28
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.

trees/tiles: Add TileReader and ReadTiles

3 participants