@kevinmcconnell please let me know if there's anything I can do to help here 🙏

Hi @3v0k4, thanks for handling this! (And sorry for my slow response).

I think since Thruster is still young, and pre-1.0, we should drop support for the old env var, rather than support it as a deprecated option. That would simplify the change, and the tests. (Although I appreciate you taking the time to add that!). At this stage of the project I think it's better to arrive at the right API, rather than accrue the baggage of the decisions that we went through to get there, if you see what I mean. It's a breaking change, but it's a very easy one to accommodate. And we can make sure to call it out clearly in the changelog.

Does that sound good to you?

Does that sound good to you?

Sounds great. I'm taking care of it.

@kevinmcconnell updated the code.

That's perfect, thanks again @3v0k4!

@kevinmcconnell Thank you!

This was my first Go OSS contribution and it's an honor to have done it paying tribute to Ruby/Rails.

Looking forward for more opportunities to contribute to your work 🙂