Donal McBreen
Donal McBreen
This was fixed in https://github.com/basecamp/kamal/pull/1229
@federicoaldunate - thanks for the PR! I'll get this merged if you could resolve the merge conflicts first.
Closed as we've merged #1229 which included these changes
Fixed by https://github.com/basecamp/kamal/pull/1508
@jeremy - I'll get the pending changes in master out in an release, and then release this on its own so the behaviour change isn't lost in a sea of...
Going to close this one, as https://github.com/basecamp/kamal/pull/1355 gives you the ability to deploy with a local registry (though it runs on the deploy host, not the application one)
> I also just tried loading `Dotenv.load(".kamal/secrets.staging", overwrite: true)` in _config/deploy.staging.yml_ but that code doesn't actually run. I would expect this to run - is the problem that `deploy.yml` is...
Thanks @nickhammond - that's all makes sense, and I think making `KAMAL_DESTINATION` available makes sense 👍
Hey @nickhammond - I'm keen to get this into Kamal - I'll be away next week, but when I'm back I'll do a final review. Thanks for your patience!
Hey @nickhammond - no nothing needed from me. I'm happy to include this in Kamal 2.7 when that is ready to go out