fix: Use localstack/localstack-pro by default

[aidehn]

Changes

• Updates the code to no longer use localstack/localstack and opt to use localstack/localstack-pro instead. The prior can still be used if specified.
• The workflow doesn't work on latest due to an issue with the Docker SDK which makes me believe it has been broken for quite a while since our testing workflow usually tests against Pro by default.
  • Since it's not been raised it is likely an issue specific to our test set up.

[alexrashed]

Thanks a lot for jumping on this! I really am a bit puzzled by the failing pipeline, but I agree that the pipeline is just broken (which isn't surprising given that this project hasn't seen much traction / maintenance lately).

[alexrashed]

question: I guess this change will potentially break the usage of the library for some consumers (i.e. those who currently use it without an auth token). Maybe this change would be worth doing a minor bump and also reference the blog post for the image consolidation here? WDYT?

[aidehn]

I think that makes sense! I'll make this a minor version bump and add some reference to the blog post 👍🏼

[alexrashed]

nit: Given that this command can never have worked, I would actually opt to just remove it.

[aidehn]

Sounds good!