Skip to content

fix missed case for nullable enums#3043

Merged
jycor merged 1 commit intomainfrom
james/enum
Jun 25, 2025
Merged

fix missed case for nullable enums#3043
jycor merged 1 commit intomainfrom
james/enum

Conversation

@jycor
Copy link
Copy Markdown
Contributor

@jycor jycor commented Jun 25, 2025

This PR adds a missed edge case from this: #2985
We were incorrectly making nullable enum columns take the first value.

Copy link
Copy Markdown
Contributor

@fulghum fulghum left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@jycor jycor merged commit 2f1daeb into main Jun 25, 2025
9 checks passed
@jycor jycor deleted the james/enum branch June 25, 2025 18: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.

2 participants