[
Date Prev][
Date Next][
Thread Prev][
Thread Next][
Date Index][
Thread Index]
[
List Home]
[automotive-pmc] Fwd: [tractusx-dev] Committer Election for Lucas Capellino on Eclipse Tractus-X has started
|
FYI
-------- Forwarded Message --------
Hey all,
Sorry for the spam, the voting page doesn't do formatting
properly, so let me elaborate on my -1 a bit:
Lacking personal experience with Lucas I can only go by what was
presented in the nomination. To me most PRs are of somewhat
reasonable size (except one or two that seem a bit unfocused),
but I obviously did not look at the PRs in terms of code
quality. I did however notice that most PRs don't have a ton of
test code which makes me question the "high software
quality standards" statement. I am a stickler for
tests, so that just might be me.
There are other issues I see, specifically:
- many PRs have no description or reviews on them
- some PRs seem to have been created by someone else, but
are still listed. not sure how they are relevant.
- PRs don't have any labels, linked issues or other metadata
on them, which (to me) would demonstrate proper OSS
procedures.
- many PRs have merge commits and commits from other
contributors on them. To me that indicates poor PR hygiene
and focus.
- I don't know what "reviews and
harmonization" means, because this bullet point
only links the repos, so I'm having a hard time
understanding the contributions there.
Overall I see considerable room for improvement in terms of OSS
procedures and transparency, and I am very sorry to say that as
things stand I cannot +1 this for the reasons stated above.
Best,
Paul
Hi
all,
first of all thank you for this
proposal. It is good to see continuous interest in
becoming Eclipse Tractus-X committer.
I want to highlight some thinks, that I
find odd on this proposal though:
In general, I find it hard to see the
actual contributions made by Lucas. A lot of PRs
contain commits done by multiple people, like
https://github.com/eclipse-tractusx/traceability-foss/pull/751
for example.
There is even one without any contributions:
https://github.com/eclipse-tractusx/traceability-foss/pull/826 This
one is especially odd, since the commits + review
(LGTM) is done by the same person. Only the PR was
created by Lucas.
I also cannot
really see the “review and harmonization” effort.
Could you maybe make this a bit more tangible?
Cheers
Sebastian
If you are not the addressee, please inform us
immediately that you have received this e-mail
by mistake, and delete it. We thank you for your
support.
|
_______________________________________________
tractusx-dev mailing list
tractusx-dev@xxxxxxxxxxx
To unsubscribe from this list, visit
https://accounts.eclipse.org
_______________________________________________
tractusx-dev mailing list
tractusx-dev@xxxxxxxxxxx
To unsubscribe from this list, visit https://accounts.eclipse.org