TODO (listx): Combine HTTPS, HTTP, and SSH schemes into a single enum.

[tsj-30]

Currently, the listx implementation treats HTTP, HTTPS, and SSH schemes as separate cases. This leads to duplicated logic, scattered conditionals, and makes it harder to extend or maintain scheme-related behavior.

There is a TODO in the codebase indicating that these schemes should be combined into a single enum, but this refactor has not yet been implemented.
// TODO (listx): Combine HTTPS, HTTP, and SSH schemes into a single enum. in prow/pkg/git/v2/client_factory.go
Proposed Change

Introduce a single enum (e.g. SchemeType) to represent supported schemes:

• HTTP

• HTTPS

• SSH

• Replace existing scheme-specific branching logic with enum-based handling

I’m happy to work on this issue and submit a PR if this approach is acceptable. Please let me know if there are any preferred patterns or constraints to follow.

cc: @petr-muller , @matthyx

[tsj-30]

/assign

[matthyx]

@tsj-30 I don't have anything specific in mind, let's see how it looks once you have something and iterate on that :)
Maybe @petr-muller has some requirements?

[tsj-30]

Sure @matthyx , Will draft a PR and make changes as we go.
Thank you 😄

[petr-muller]

I do not have any specific insights about that unfortunately, haven't seen that code for a while. Take a shot at it and let's see where it goes 👍