Pygithub: Добавлена ​​поддержка расширенных API-интерфейсов защиты ветвей.

@harlowja Я сделал немного этого в https://github.com/a13m/PyGithub/commit/3af108372a5179e16fc093b78568c8e45a3a91ca

Мне хватило хотя бы провести аудит защиты нашей ветки. YMMV.

Да, было бы здорово, если бы что-то подобное объединили (и выпустили); эта библиотека, кажется, отстает от фактического API и/или возможностей, что меня немного огорчает :-/

Особенно хотелось бы этого, поскольку я пытался убедиться, что вся моя конфигурация github для моей группы находится в формате yaml, а не скрыта в пользовательском интерфейсе github, а затем имеет программу, которая анализирует указанный формат yaml и настраивает различные репозитории github. Это создает преемственность (и здравомыслие) в репозиториях моих групп, а возможность программной настройки защиты рецензентов логически была бы чем-то, что команда хотела бы сделать (чтобы список рецензентов можно было контролировать и управлять с помощью аналогичного формата yaml).

Хорошая работа @a13m!
Тем временем: https://developer.github.com/changes/2017-09-06-protected-branches-preview-end/.

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

теперь вам необходимо передать объект required_pull_request_reviews при вызове конечной точки защиты ветки обновления. Передача этого объекта была необязательной в период предварительной версии.

Я бы тоже хотел это увидеть. Пиарите пожалуйста ❤️

На самом деле это большой бардак. Repository.protect_branch() обращается к конечной точке, которая, вероятно, была удалена, так как GitHub написал в блоге, что это произойдет после выпуска Branch Protection. Мне было интересно, почему документы API и код полностью расходятся друг с другом.

Я собираюсь добавить новый метод Repository.edit_branch_protection() для вызова новой конечной точки с новыми структурами данных, и у меня возникает соблазн одновременно удалить Repository.protect_branch() из-за поломки.

Привет, ребята, мой патч еще не завершен, но я подумал, что было бы неплохо получить ранний отзыв ^

Я также работал над этим, подходя к этому по-другому — добавляя методы в Branch и удаляя Repository.protect_branch().

@stevenk ах, теперь я это вижу. Хм, не хочу дублировать работу. Мое намерение состояло в том, чтобы сохранить API библиотеки прежним. Как вы хотите это сделать?

@JPWKU Извините, я должен был более четко сообщить, что активно работаю над этим. Метод Repository.protect_branch() действительно соответствовал GitHub API, пока они не удалили его, когда новый API защиты ветвей был выпущен.

Мой текущий патч реализует API защиты веток, как показывает GitHub на https://developer.github.com/v3/repos/branches/ , как следствие, он довольно большой. У меня есть 3 или около того тестовых случаев, которые нужно закончить, прежде чем я подниму это.

@stevenk не беспокойтесь, так как у вас уже есть работа в этой области, я закрою свой PR и прокомментирую/помогу вам.

@stevenk Планируете ли вы объединить это в ближайшее время? Мне нужен этот функционал.

@prburgu Не от меня зависит, когда он сливается. @JPWKU , когда у вас есть время, чтобы продолжить просмотр прикрепленного PR?

@prburgu @stevenk Я постараюсь выкроить время сегодня или вечером, чтобы закончить обзор.

Есть новости по этому поводу?