Pygithub: Tambahkan dukungan untuk API perlindungan cabang tingkat lanjut

@harlowja Saya melakukan sedikit ini di https://github.com/a13m/PyGithub/commit/3af108372a5179e16fc093b78568c8e45a3a91ca

Sudah cukup bagi saya untuk setidaknya mengaudit perlindungan cabang kami. YMMV.

Ya, akan sangat bagus jika sesuatu seperti itu digabungkan (dan dirilis); perpustakaan ini tampaknya tertinggal di belakang API dan/atau kemampuan yang sebenarnya, yang membuat saya sedikit sedih :-/

Terutama akan seperti ini karena saya sudah mencoba untuk memastikan semua konfigurasi github saya untuk grup saya dalam format yaml, dan tidak disembunyikan di UI github, dan kemudian memiliki program yang mem-parsing format yaml tersebut dan menyesuaikan berbagai repositori github. Ini menciptakan kesinambungan (dan kewarasan) di seluruh repositori grup saya dan memiliki kemampuan untuk menyesuaikan perlindungan pengulas secara terprogram secara logis akan menjadi sesuatu yang ingin dilakukan tim (sehingga daftar pengulas dapat dikontrol & dikelola melalui format yaml yang serupa).

Kerja bagus @a13m!
Sementara itu: https://developer.github.com/changes/2017-09-06-protected-branches-preview-end/.

Sepertinya kita terjebak sampai seseorang mengumpulkan energi untuk mengatasi ini.

Anda sekarang harus melewati objek required_pull_request_reviews saat memanggil titik akhir perlindungan cabang pembaruan. Melewati objek ini adalah opsional selama periode pratinjau.

Saya akan senang melihat ini juga. Tolong PRnya ❤️

Ini sebenarnya kekacauan besar. Repository.protect_branch() memanggil ke titik akhir yang mungkin telah dihapus, karena GitHub membuat blog bahwa itu akan terjadi ketika Perlindungan Cabang lulus. Saya bertanya-tanya mengapa dokumen API dan kodenya benar-benar bertentangan satu sama lain.

Saya akan menambahkan metode baru Repository.edit_branch_protection() untuk memanggil titik akhir baru dengan struktur data baru, dan saya tergoda untuk menghapus Repository.protect_branch() pada saat yang sama karena rusak.

Hai teman-teman, tambalan saya belum lengkap tetapi saya pikir akan menyenangkan untuk memiliki umpan balik awal ^

Saya juga telah mengerjakan ini, mendekatinya dengan cara yang berbeda -- dengan menambahkan metode ke Branch, dan menghapus Repository.protect_branch()

@stevenk ah saya melihat itu sekarang. Hmm tidak ingin menduplikasi pekerjaan. Niat saya adalah untuk menjaga api perpustakaan tetap sama. Bagaimana Anda ingin melakukannya?

@JPWKU Maaf, saya seharusnya mengomunikasikan lebih jelas bahwa saya aktif mengerjakan ini. Metode Repository.protect_branch() memang cocok dengan GitHub API, sampai mereka menghapusnya saat API perlindungan cabang baru lulus.

Tambalan saya saat ini mengimplementasikan API perlindungan cabang seperti yang ditunjukkan GitHub di https://developer.github.com/v3/repos/branches/ , sebagai akibatnya, ini agak besar. Saya memiliki 3 atau lebih kasus uji untuk menyelesaikan penulisan sebelum saya mendorong ini.

@stevenk jangan khawatir, karena Anda sudah mendapat pekerjaan di bidang ini, saya akan menutup PR saya dan mengomentari/membantu Anda.

@stevenk Apakah Anda berencana untuk menggabungkan ini segera? Saya membutuhkan fungsi ini.

@prburgu Terserah saya ketika bergabung. @JPWKU kapan ada waktu untuk melanjutkan review PR terlampir?

@prburgu @stevenk Saya akan mencoba mengukir waktu hari ini atau malam ini untuk menyelesaikan ulasan.

Ada pembaruan tentang ini?