This is kind of a breaking change, but actually the code was wrong before; sa_column always overrides sqlmodel plumbed kwargs, so the nullable=False outside of the sa.Column was being thrown away (as evidenced by the resulting schema change in local_auth).
sqlmodel now raises an exception when you pass props that will be ignored, and these apps were both doing it wrong
nullablehas to be set on the column.This is kind of a breaking change, but actually the code was wrong before;
sa_columnalways overrides sqlmodel plumbed kwargs, so thenullable=Falseoutside of thesa.Columnwas being thrown away (as evidenced by the resulting schema change inlocal_auth).sqlmodel now raises an exception when you pass props that will be ignored, and these apps were both doing it wrong