NO TRANSACTION annotation is case sensitive

[visortelle]

According to the documentation, annotations are case-insensitive: https://pressly.github.io/goose/reference/annotations/

However, I wasn't able to get rid of the CREATE INDEX CONCURRENTLY cannot run inside a transaction block error without setting the NO TRANSACTION annotation in upper-case.

[mfridman]

Which version of goose are you using?

The case sensitive annotations were added in v3.19.1.

[visortelle]

github.com/pressly/goose v2.7.0+incompatible // indirec

github.com/pressly/goose v2.7.0+incompatible h1:PWejVEv07LCerQEzMMeAtjuyCKbyprZ/LBa6K5P0OCQ=
github.com/pressly/goose v2.7.0+incompatible/go.mod h1:m+QHWCqxR3k8D9l7qfzuC/djtlfzxr34mozWDYEu1z8

[visortelle]

Ah, ok. It's already 3.20.x. Let me update and check.

[visortelle]

Thanks @mfridman
Confirm, that it works as expected on v3.
Closing.

[mfridman]

It'd be nice if we added the minimum version throughout the docs (which I'm currently working to expand).

But it's a bit of overhead to track, so not sure we'll end up doing it due to maintenance cost.

[mfridman]

Also, I just noticed you mentioned going from v2.7.0+incompatible to 3.20.x. and it just worked?

We tried really hard to keep backwards compatibility so I'm glad nothing broke for you!

[visortelle]

@mfridman Don't worry, I use goose in a new project that doesn't have a live database yet.
In v3 the no transaction annotation works in both upper and lower cases, so it's backward compatible.
Thanks for taking care!