regro / cf-scripts

Flagship repo for cf-regro-autotick-bot
Other
46 stars 70 forks source link

Readd git backend #2778

Closed ytausch closed 3 weeks ago

ytausch commented 3 weeks ago

This reapplies #2643 after #2705 reverted the change. Turns out the default connection mode must be HTTPS, not SSH.

beckermr commented 3 weeks ago

Should we remove the ssh urls all together? I don't expect we'll ever use them.

ytausch commented 3 weeks ago

Good point, I'll do that later

beckermr commented 3 weeks ago

Shall we try to merge again and do it in another PR or do you want to do it here?

ytausch commented 3 weeks ago

I've done it here, should work now.

I did not remove the ConnectionMode concept completely because I think it's a sensible abstraction for the future still.