feat(analyser): banDropNotNull rule

[juleswritescode]

I tried adding a new lint rule, but the just gen-lint script fails:

I can't see a syntax error in this statement:
alter table users alter column email drop not null;

so maybe something is broken?

[juleswritescode]

will check this, too – my local prettier adds an extra ` for rust

[juleswritescode]

Ha, this is actually makes sense, because there are triple backticks within the code block. 👍

[psteinroe]

Thanks for working through the guide! I will check what's wrong. We should also add a test framework (like biome) to quickly run the rules over a set of sql files and compare expected results. 🙏🏼

[psteinroe]

This could even be an issue in the statement splitter 😅 it seems like we are cutting the statement too early.

[juleswritescode]

This could even be an issue in the statement splitter 😅 it seems like we are cutting the statement too early.

Ah just realized we're currently ignoring the banDropColumn sql. That's why it worked with the existing rule :)

I'll just ignore the SQL block in the new rule for now.

[juleswritescode]

Thanks for working through the guide! I will check what's wrong. We should also add a test framework (like biome) to quickly run the rules over a set of sql files and compare expected results. 🙏🏼

100%, I can make that my next priority :) 👍

[psteinroe]

Thanks!