Temurin-build: Изучите пиар-крючок, чтобы проверить, подписал ли кто-то ОСА, и если не побуждать его

Созданный на 30 мар. 2017  ·  5Комментарии  ·  Источник: adoptium/temurin-build

Это сделано для того, чтобы, если бы мы захотели пожертвовать какой-либо рабочий код проекту OpenJDK, мы могли бы это сделать.

Это может работать так же, как бот MS PR для этого PR -> https://github.com/Azure/azure-iot-sdks/pull/478

Мы проделали некоторую раннюю работу над предыдущей инкарнацией фермы сборки OpenJDK (Betterrev), чтобы получить список подписантов OCA с веб-сайта openjdk (скребок экрана HTML), и поэтому я думаю, что мы можем использовать это для проверки адреса электронной почты пользователя GitHub (если доступно в API).

Это должно применяться ко всем репозиториям openjdk- *

Также следует приложить усилия, чтобы гарантировать, что все существующие дополнения кода до того, как этот PR-крючок будет введен в действие, могут быть пожертвованы проекту OpenJDK в более поздний срок, убедившись, что коммиттеры подписали OCA.

Возможно, мне здесь не хватает некоторых крайних случаев / разъяснений - дополнительные комментарии приветствуются

help wanted invalid
Источник
picture karianna

Все 5 Комментарий

Страница подписантов OCA не содержит идентификаторов электронной почты или github для всех из них - например, для нашей записи это только мое имя и организация: (пример из http://www.oracle.com/technetwork/community/oca- 486395.html # g)

   Goldman Sachs - OpenJDK (Sunny Chan, Jessica Man)

С некоторыми из них действительно связана учетная запись github в другом формате:

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

Хотя у некоторых есть идентификатор java.net:

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

Поэтому я не уверен, сколько вы можете снимать с экрана (ну, я не видел кода скрапа с экрана betterrev)

Возможно, лучший способ сделать это - перед тем, как принять запрос на извлечение, мы потребуем от пользователя ввести определенную строку, которая соответствует записи (например, OCA: Goldman Sachs) и вашему скрипту, чтобы оно соответствовало имени. Если строки OCA нет, отправьте им электронное письмо.

sunnychanwork picture sunnychanwork 7 апр. 2017
👍1

Я согласен с @sunnychanwork , очистка списка OCA будет ограниченным преимуществом.
Я предлагаю просто провести ручную проверку для каждого нового участника и позволить боту поддерживать наш собственный список идентификаторов github, которые являются известными подписантами OCA для сравнения.

Если нас перегружает большое количество участников, тогда (это было бы неплохо, и) мы можем придумать дальнейшую автоматизацию, но я не ожидаю, что это станет непосредственной проблемой.

tellison picture tellison 7 апр. 2017
👍1

Изменилось ли это требование и осталось ли оно обязательным? А как насчет тех, кто не подписал (или не может) OCA, если эти взносы не будут разрешены?

Насколько я понимаю, openjdk-build и openjdk-Infrastructure - это своего рода отличия от того, что они напрямую связаны с изменениями исходного кода openjdk. Должны ли они подпадать под одно и то же требование?

jdekonin picture jdekonin 19 мар. 2018

Я использую Iceboxing, поскольку (на данный момент) мы на самом деле не действуем как система отправки патчей для апстрима.

karianna picture karianna 19 мар. 2018

Не требуется, так как OCA подписаны для получения обновлений.

karianna picture karianna 10 дек. 2018
Была ли эта страница полезной?
0 / 5 - 0 рейтинги

Смежные вопросы

sxa picture sxa  ·  4Комментарии
karianna picture karianna  ·  7Комментарии
lumpfish picture lumpfish  ·  4Комментарии
pshipton picture pshipton  ·  4Комментарии
ChristianCiach picture ChristianCiach  ·  7Комментарии