ColumnDefinitions were previously being output however they were done completely within the TableDefinition. This caused a few issues in regards to log warnings as well as complicated the update process for existing columns.
This change teases this logic apart so that the column logic exists primarily within a column structure. The only exception to this is the removal of columns due to no object existing - this is still completed on the TableDefinition.
Amongst all of this, I updated lalrpop to 0.15, fixed the compat issues and removed superfluous warnings. There are warnings remaining about unused TableConstraints however these will be addressed with #30.
Closes #29
ColumnDefinition
s were previously being output however they were done completely within theTableDefinition
. This caused a few issues in regards to log warnings as well as complicated the update process for existing columns.This change teases this logic apart so that the column logic exists primarily within a column structure. The only exception to this is the removal of columns due to no object existing - this is still completed on the
TableDefinition
.Amongst all of this, I updated
lalrpop
to0.15
, fixed the compat issues and removed superfluous warnings. There are warnings remaining about unusedTableConstraint
s however these will be addressed with #30.