Skip to content

Conversation

@ncruces
Copy link

@ncruces ncruces commented Dec 15, 2025

You may have missed that sqlite3_table_column_metadata gets additional metadata for columns, in particular NOT NULL constraints.

Unfortunately because of backwards compatibility a PRIMARY KEY column is not always NOT NULL, and it's really hard to correctly detect INTEGER PRIMARY KEY columns which are NOT NULL.

So this misses a few NOT NULL columns, but it's better than nothing, I guess.

@dosubot dosubot bot added size:S This PR changes 10-29 lines, ignoring generated files. 🔧 golang labels Dec 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:S This PR changes 10-29 lines, ignoring generated files. 🔧 golang

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant