tessi / capistrano-uberspace

Deploy your rails app on an uberspace with Capistrano 3
MIT License
14 stars 7 forks source link

Uberspace 7 not supported? #15

Open jmuheim opened 5 years ago

jmuheim commented 5 years ago

I receive a lot of bash errors like:

bash: uberspace-add-domain: command not found

I suppose this is because I have an Uberspace 7 account, not a 6?

A note in the README would be useful.

tessi commented 5 years ago

Hey jmuheim,

many thanks for opening this issue! Indeed, I have not tested on uberspace 7. But the error sounds like some commands have changed with uberspace 7 🤔

I added a note to the README. If you have any hints or a PR ready on how to make this gem uberspace7-ready, I'd be glad to hear them :)

tessi commented 5 years ago

I will leave this issue open to collect hints on potential fixes and give this issue more visibility.

mamhoff commented 4 years ago

Hey @tessi :) I have a version of this that does support U7 right now, it's probably just a matter of splitting the work up into meaningful chunks for PRs: https://github.com/mamhoff/capistrano-uberspace

tessi commented 4 years ago

hey. @mamhoff :) nice to see from you - especially with progress on the issue.

My free time is currently very constrained and my work-time doesn't allow working on this. So we either have to wait for a free weekend on my side or some PRs from the community (or you) to review.

I think this gem probably supports too many options (commands, db adapters) to migrate them all in one chunk. I guess we should introduce a new mayor version with breaking changes and bit-by-bit upgrade the things that people still want to uberspace 7. What do you think?

I'm also open to accept more contributors on this repository, since this situation is a typical bus-factor issue which can only be solved by having more people to rely on.