cloudspannerecosystem / memefish

memefish is the foundation to analyze Spanner SQL
https://cloudspannerecosystem.dev/memefish/
MIT License
76 stars 19 forks source link

Sort `Statement` and `DDL` types #160

Closed makenowjust closed 3 weeks ago

makenowjust commented 3 weeks ago

This order follows the official documentation:

If this rule can be enforced by a linter, it sounds really good.

apstndb commented 3 weeks ago

If this rule can be enforced by a linter, it sounds really good.

I have no idea, but organizing types in ast.go into sections may be a good starting point .

e.g.

// ================================================================================
//
// Table statements
//
// ================================================================================

// This section only contains TABLE related nodes like `CreateTable`, `DropTable`, `AlterTable`, `RenameTable`
...