Skip to content

fix: prevent Write(nil) from panicking - #5

Merged
masonkatz merged 2 commits into
mainfrom
fix/write-nil-panic
Aug 22, 2026
Merged

fix: prevent Write(nil) from panicking#5
masonkatz merged 2 commits into
mainfrom
fix/write-nil-panic

Conversation

@masonkatz

Copy link
Copy Markdown
Member

Summary

  • guard against nil reflection types in Table.Write
  • add a regression test for Write(nil)

Fixes #1

@masonkatz
masonkatz force-pushed the fix/write-nil-panic branch from 52b6ef8 to fcdbdcf Compare August 22, 2026 01:06
@codecov

codecov Bot commented Aug 22, 2026

Copy link
Copy Markdown

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

masonkatz and others added 2 commits August 21, 2026 18:28
Fixes #1

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@masonkatz
masonkatz force-pushed the fix/write-nil-panic branch from 575afd4 to 32885a2 Compare August 22, 2026 01:28
@masonkatz
masonkatz merged commit d656587 into main Aug 22, 2026
4 checks passed
@masonkatz
masonkatz deleted the fix/write-nil-panic branch August 23, 2026 00:12
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.

fix: prevent Write(nil) from panicking

1 participant