fix: remove not function that expects two arguments

[sanjibansg]

This PR removes the not function in functions_boolean.yaml that expects two boolean arguments.

[cpcloud]

LGTM

[jacques-n]

I think we need this commit to be marked BREAKING CHANGE according to the semantic release plugin since removal of a function is a breaking change. See here.

@cpcloud , any chance you can point to a good example of what a commit message should look like?

[cpcloud]

Here's an example from ibis: ibis-project/ibis@3f48cb8. The commit message is:

refactor(api): remove distinct ColumnExpr API

Remove `ColumnExpr.distinct()` because it's a confusing API.

BREAKING CHANGE: Replace `t["a"].distinct()` with `t[["a"]].distinct()`.

[cpcloud]

@jacques-n How do you want to proceed here?

[jacques-n]

@jacques-n How do you want to proceed here?

Isn't the commit message still missing the breaking change notice? Frankly, I don't think anyone is depending on this change so I'm not stuck on calling it a breaking change other than for rigor.

[sanjibansg]

I have updated the commit message to include BREAKING CHANGE, does this look good now?

[westonpace]

The commit message looks correct to me now.

[cpcloud]

Sorry, I meant to block accidental merging until #210 is merged :)

[jacques-n]

Now that #210 is merged, merging this.

#210 is now merged.