woodpecker-ci / woodpecker

Woodpecker is a simple, yet powerful CI/CD engine with great extensibility.
https://woodpecker-ci.org
Apache License 2.0
4.3k stars 371 forks source link

Support Git as only VCS #4346

Open qwerty287 opened 1 week ago

qwerty287 commented 1 week ago

Only support git as vcs and remove the corresponding env var CI_REPO_SCM.

6543 commented 1 week ago

why was this initially proposed at all? e.g. what's the reason behind?

6543 commented 1 week ago

I see the current architecture that we can support more than git even as far as an "selling" over other cicd systems

xoxys commented 1 week ago

e.g. https://github.com/woodpecker-ci/woodpecker/discussions/2355 integration

How are other git forges that were implemented by an external plugin related to removing none-git vsc systems?

I see the current architecture that we can support more than git even as far as an "selling" over other cicd systems

I don't see it as a selling point especially not as e can't even handle all existing bugs for the existing feature set already.

why was this initially proposed at all?!? e.g. what's the reason behind?

I would like to kindly ask you again not to communicate with so many punctuation marks. It's hard to read and a somewhat “aggressive” tone.

6543 commented 1 week ago

PS: still thanks for thinking how to slim down and remove legacy code :)

qwerty287 commented 1 week ago

I don't see it as a selling point especially not as e can't even handle all existing bugs for the existing feature set already.

That's also my point. Radicle has a git "bridge" so it should work.

And we can't have everything as selling point. At some point, we should have some limitations, we can't build the "egg-laying woolly milk sow" (don't know if there's a proper English saying 😅).

6543 commented 1 week ago

we can't build the "egg-laying woolly milk sow"

agree but this is realy not much code and losing it is a shame ...

6543 commented 1 week ago

I as an owner wana veto this change

pat-s commented 6 days ago

Without any opinion: Should this PR be closed then?

xoxys commented 6 days ago

If a single owner has the power to block PRs at any time, yes.

qwerty287 commented 6 days ago

Well, @anbraten approved it...

woodpecker-bot commented 5 days ago

Deployment of preview was successful: https://woodpecker-ci-woodpecker-pr-4346.surge.sh

6543 commented 5 days ago

well the argument stands: either merge it, in this case I will also merge things, that got denyed from me, because an owner is for it ... or block it untill veto is removed in this case untill I'm no owner anymore. next election is in 1 months btw. ...

so yes our community gudelines dont have a clear statement of when the owners dissagree - wich would not be a problem if we would have $owner.mod(2) = 0 (and if there is something stated as majority of owners agree ..), .. witch we currently dont.

so I personally interpret it as veto. ?

xoxys commented 4 days ago

I just dont understand your point. Right now there is not a single none-git forge supported in woodpecker core right? So this code is unused and useless. To keep it just because we might or might not support none-git forges doesnt make sense to me. Even if we would like to support such forges we could also do it by an official extension in the same way we do it for plugins.

pat-s commented 3 days ago

As this is now more a political issue than a code-one, it would be great if you @6543 @anbraten could discuss the general governance part how to proceed in such situations.

@qwerty287 I think there's not much value in further branch updated until this is resolved?

qwerty287 commented 3 days ago

Tbh I personally think owners should only exist because they're necessary, but they shouldn't have additional permissions like a veto. All maintainers should have the same possibilities. If the maintainers can't come to a conclusion, it's fine that an owner's vote counts more or something like this. You still need owners of course because we have servers and online accounts etc. And thus I also think the name "owners" is not the best, "admins" would be better.