These two regex won't match with the complex column type now.

[haudoing]

#783

[CLAassistant]

All committers have signed the CLA.

[SpencerTorres]

Thanks for the issue + PR!

I see Nullabe(IP) was also changed, is this covered by a test case? I see theres some other ones in there but not specifically for array nullable.

[haudoing]

Thanks for the issue + PR!

I see Nullabe(IP) was also changed, is this covered by a test case? I see theres some other ones in there but not specifically for array nullable.

Please take a look. I created another testcase to cover column type "Array(Nullable(IPv4))"

[SpencerTorres]

Just noticed there's IPv4 and IPv6, could you add one more test case for IPv6?

Other than that, the PR looks good. I'll run it locally to verify

[SpencerTorres]

"Nullable(IP)":              regexp.MustCompile(`Nullable\(IP`),
"Nullable(String)":          regexp.MustCompile(`Nullable\(String`),

It's very interesting that of all of the types there, these are the only two without the ^ prefix. I was worried this was intentional, but after seeing the original commits for String and IP, it seems this was just a small oversight.

I've run the tests and everything looks good. Thanks for the contribution 👍

[bossinc]

Thank you @haudoing and @SpencerTorres !