Temurin-build: Untersuchen Sie einen PR-Hook, um zu überprüfen, ob jemand die OCA unterzeichnet hat und ob Sie ihn nicht dazu auffordern

Erstellt am 30. März 2017 · 5Kommentare · Quelle: adoptium/temurin-build

Dies ist so, dass wir, wenn wir funktionierenden Code an das OpenJDK-Projekt spenden wollten, dies tun könnten.

Dies könnte ähnlich wie der MS PR-Bot für diesen PR funktionieren --> https://github.com/Azure/azure-iot-sdks/pull/478

Wir haben einige frühe Arbeiten an einer früheren Inkarnation der OpenJDK-Buildfarm (Betterrev) durchgeführt, um eine Liste der OCA-Unterzeichner von der openjdk-Website (HTML-Screenscraper) abzurufen, und ich denke, wir können damit die E-Mail-Adresse des GitHub-Benutzers überprüfen (wenn in der API verfügbar).

Dies sollte auf alle openjdk-*-Repositorys angewendet werden

Es sollte auch versucht werden sicherzustellen, dass alle vorhandenen Codebeiträge vor der Einrichtung dieses PR-Hooks zu einem späteren Zeitpunkt an das OpenJDK-Projekt gespendet werden können, indem sichergestellt wird, dass die Committer die OCA unterzeichnet haben.

Ich vermisse hier möglicherweise einige Grenzfälle / Klarstellungen - zusätzliche Kommentare willkommen

help wanted invalid

Quelle

karianna

Alle 5 Kommentare

Die OCA-Unterzeichnerseite enthält nicht für alle eine E-Mail- oder Github-ID - zum Beispiel ist es für unseren Eintrag nur mein Name und meine Organisation: (Beispiel von http://www.oracle.com/technetwork/community/oca- 486395.html#g)

   Goldman Sachs - OpenJDK (Sunny Chan, Jessica Man)

Einige von ihnen haben tatsächlich ein damit verbundenes Github-Konto in einem anderen Format:

   Adolfo Dos Santos Jr - Jersey - GitHub adolfojunior
   Alessandro Gherardi - Jersey - GH agherardi

Während einige eine java.net-ID haben:

 Mohammad W. Abdo - JDK - java.net - mohammad

Ich bin mir also nicht sicher, wie viel Sie Schrott anzeigen können (nun, ich habe den betterrev-Bildschirmschrottcode nicht gesehen)

Vielleicht ist es besser, dass wir vor dem Akzeptieren der Pull-Anfrage den Benutzer auffordern, eine bestimmte Zeile einzugeben, die dem Eintrag entspricht (zB OCA: Goldman Sachs) und Ihr Skript zum Namen passt. Wenn die OCA-Leitung nicht da ist, senden Sie ihnen eine E-Mail.

sunnychanwork am 7. Apr. 2017

👍1

Ich stimme @sunnychanwork zu , es wird einen begrenzten Vorteil geben, die OCA-Liste zu kratzen.
Ich schlage vor, dass wir einfach für jeden neuen Beitragenden eine manuelle Überprüfung durchführen und den Bot unsere eigene Liste von Github-IDs führen lassen, die als OCA-Unterzeichner bekannt sind, mit denen verglichen werden kann.

Wenn wir mit vielen Mitwirkenden überwältigt sind (das wäre ein schönes Problem und) können wir eine weitere Automatisierung finden, aber ich erwarte nicht, dass dies ein unmittelbares Problem ist.

tellison am 7. Apr. 2017

👍1

Hat sich diese Anforderung geändert und ist sie noch immer erforderlich? Was ist mit denen, die das OCA nicht unterschrieben haben (oder können), sollten diese Beiträge nicht zugelassen werden?

openjdk-build und openjdk-infrastructure sind, wie ich es verstehe, irgendwie unabhängig davon, dass sie direkt an Openjdk-Quelländerungen gebunden sind. Sollten diese unter die gleiche Anforderung fallen?

jdekonin am 19. März 2018

Ich mache das Iceboxing, da wir (im Moment) nicht wirklich als Patch-Einreichungssystem für Upstream fungieren

karianna am 19. März 2018

Nicht erforderlich, da OCAs für Upstream-Patches signiert sind.

karianna am 10. Dez. 2018

War diese Seite hilfreich?

0 / 5 - 0 Bewertungen

Temurin-build: Untersuchen Sie einen PR-Hook, um zu überprüfen, ob jemand die OCA unterzeichnet hat und ob Sie ihn nicht dazu auffordern

Alle 5 Kommentare

Verwandte Themen