Skip to content

fix: Builder and Reader initialized checks - #155

Merged
tmathern merged 4 commits into
mainfrom
mathern/init-states
Sep 3, 2025
Merged

fix: Builder and Reader initialized checks#155
tmathern merged 4 commits into
mainfrom
mathern/init-states

Conversation

@tmathern

@tmathern tmathern commented Aug 29, 2025 •
edited
Loading

Copy link
Copy Markdown
Collaborator
  • Improve internal state checks
  • Make sure no Reader or Builder is partially initialized (in case of failures during construction)

@tmathern tmathern self-assigned this Aug 29, 2025
@tmathern
tmathern requested review from dyro, gpeacock and ok-nick August 29, 2025 20:11
@tmathern
tmathern merged commit 0d369fa into main Sep 3, 2025
20 checks passed
@tmathern
tmathern deleted the mathern/init-states branch September 3, 2025 19:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@gpeacock gpeacock gpeacock approved these changes
@dyro dyro Awaiting requested review from dyro
@ok-nick ok-nick Awaiting requested review from ok-nick

Assignees

@tmathern tmathern

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@tmathern @gpeacock