Temurin-build: Selidiki kait PR untuk memeriksa apakah seseorang telah menandatangani OCA dan jika tidak meminta mereka

Dibuat pada 30 Mar 2017 · 5Komentar · Sumber: adoptium/temurin-build

Ini agar jika kami ingin menyumbangkan kode kerja apa pun ke proyek OpenJDK, kami dapat melakukannya.

Ini bisa bekerja seperti bot MS PR yang terhubung untuk PR ini -> https://github.com/Azure/Azure-iot-sdks/pull/478

Kami melakukan beberapa pekerjaan awal pada inkarnasi sebelumnya dari OpenJDK build farm (Betterrev) untuk menarik daftar penandatangan OCA dari situs web openjdk (pengikis layar HTML) dan jadi saya pikir kami dapat menggunakannya untuk memeriksa alamat email pengguna GitHub (jika tersedia di API).

Ini harus diterapkan ke semua repo openjdk-*

Upaya juga harus dilakukan untuk memastikan bahwa semua kontribusi kode yang ada sebelum kait PR ini diterapkan dapat disumbangkan ke proyek OpenJDK di kemudian hari dengan memastikan para pembuat komitmen telah menandatangani OCA.

Saya mungkin melewatkan beberapa kasus/klarifikasi tepi di sini - komentar tambahan diterima

help wanted invalid

Sumber

karianna

Semua 5 komentar

Halaman penandatangan OCA tidak berisi email atau id github untuk semuanya - misalnya, untuk entri kami hanya nama dan organisasi saya: (contoh dari http://www.Oracle.com/technetwork/community/oca- 486395.html#g)

   Goldman Sachs - OpenJDK (Sunny Chan, Jessica Man)

Beberapa dari mereka sebenarnya memiliki akun github yang terkait dengannya, dalam format yang berbeda:

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

Sementara beberapa memiliki id Java.net:

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

Jadi saya tidak yakin seberapa banyak Anda dapat menyaring memo (well, saya belum melihat kode memo layar yang lebih baik)

Mungkin cara yang lebih baik untuk melakukannya adalah sebelum menerima permintaan Tarik, kami akan meminta pengguna untuk meletakkan baris tertentu yang cocok dengan entri (misalnya OCA: Goldman Sachs) dan skrip Anda agar sesuai dengan namanya. Jika saluran OCA tidak ada, kirimkan email kepada mereka.

sunnychanwork pada 7 Apr 2017

👍1

Saya setuju dengan @sunnychanwork , akan ada keuntungan terbatas dari menggores daftar OCA.
Saya sarankan kita cukup melakukan pemeriksaan manual untuk setiap kontributor baru, dan biarkan bot mempertahankan daftar id github kita sendiri yang dikenal sebagai penandatangan OCA untuk dibandingkan.

Jika kita kewalahan dengan banyak kontributor maka (itu akan menjadi masalah yang bagus untuk dimiliki dan) kita dapat mengetahui otomatisasi lebih lanjut, tetapi saya tidak mengharapkan itu menjadi masalah langsung.

tellison pada 7 Apr 2017

👍1

Apakah persyaratan ini berubah dan masih diperlukan? Bagaimana dengan mereka yang belum menandatangani (atau tidak dapat) OCA, apakah kontribusi tersebut tidak diperbolehkan?

openjdk-build dan openjdk-infrastructure agak terpisah dari terikat langsung dengan perubahan sumber openjdk seperti yang saya mengerti. Haruskah ini termasuk dalam persyaratan yang sama?