search

drogue-iot / drogue-cloud

Cloud Native IoT
https://drogue.io
Apache License 2.0
114 stars 30 forks source link

Use HTTP URL for submodule as it doesn't require login to github #184

Closed dejanb closed 2 years ago

dejanb commented 2 years ago

Noticed this while testing on a new VM the other day ... git:// URL will ask you to be logged in, while https one is not

ctron commented 2 years ago

The problem with that is however, that it makes it trickier for developers (which are logged in to github) to develop. As they would need to add a new remote for pushing.

I know this can be tricky for non-developers, which clone the repository via https:// … however, I am not sure this is the primary use case.

We already had this in https://github.com/drogue-iot/drogue-cloud/pull/113 … however, it looks like we didn't track the discussion around that.

dejanb commented 2 years ago

OK, that makes sense ... I'll document it then