Pygithub: إضافة دعم لواجهات برمجة تطبيقات حماية الفروع المتقدمة

تم إنشاؤها على ٣٠ يونيو ٢٠١٧ · 15تعليقات · مصدر: PyGithub/PyGithub

يبدو أن واجهات برمجة التطبيقات التالية موجودة في github v3 (كإصدار تجريبي؟) ولكنها لم تنعكس بعد في هذه المكتبة.

https://developer.github.com/v3/repos/branches/#get -pull-request-review-إنفاذ-of-محمية-فرع
https://developer.github.com/v3/repos/branches/#update -pull-request-review-إنفاذ-of-محمية-فرع
https://developer.github.com/v3/repos/branches/#get -admin-Enfor-of-محمية-فرع
https://developer.github.com/v3/repos/branches/#get -restrictions-of-محمية-فرع
وعدد قليل من الآخرين ....

سيكون من الرائع الحصول على دعم كامل لواجهة برمجة التطبيقات هذه (خاصةً لأنها تسمح بالتحكم في مراجعات العلاقات العامة وعدد المراجعات المطلوبة وبواسطة من).

feature request high priority

مصدر

harlowja

👍5

التعليق الأكثر فائدة

هذه في الواقع فوضى كبيرة. يستدعى Repository.protect_branch () نقطة نهاية ربما تمت إزالتها ، منذ أن قام GitHub بتدوين أنه سيكون عندما تخرج حماية الفرع. لقد كنت أتساءل عن سبب تعارض مستندات API والرمز تمامًا مع بعضهما البعض.

سأقوم بإضافة طريقة جديدة Repository.edit_branch_protection () لاستدعاء نقطة النهاية الجديدة بهياكل البيانات الجديدة ، وأنا أميل إلى إزالة Repository.protect_branch () في نفس الوقت لكسرها.

s-t-e-v-e-n-k في ٣٠ أبريل ٢٠١٨

👍3

ال 15 كومينتر

harlowja فعلت القليل من هذا في https://github.com/a13m/PyGithub/commit/3af108372a5179e16fc093b78568c8e45a3a91ca

كان كافيًا بالنسبة لي على الأقل تدقيق إجراءات الحماية في فرعنا. YMMV.

a13m في ٥ يوليو ٢٠١٧

يا ، سيكون من الرائع أن يتم دمج شيء كهذا في (وإصداره) ؛ يبدو أن هذه المكتبة متخلفة عن واجهة برمجة التطبيقات الفعلية و / أو الإمكانات ، مما يجعلني حزينًا بعض الشيء: - /

harlowja في ١٢ سبتمبر ٢٠١٧

سأحب هذا على وجه الخصوص لأنني كنت أحاول التأكد من أن جميع تكوينات github لمجموعتي تكون بتنسيق yaml ، وليست مخفية بعيدًا في واجهة مستخدم github ، ثم وجود برنامج يوزع تنسيق yaml ويضبط مستودعات github المختلفة. هذا يخلق استمرارية (وسلامة) عبر مستودعات مجموعاتي ولدي القدرة على ضبط حماية المراجعين برمجيًا سيكون منطقيًا شيئًا يريد الفريق القيام به (بحيث يمكن التحكم في قائمة المراجعين وإدارتها عبر تنسيق yaml مماثل).

harlowja في ١٢ سبتمبر ٢٠١٧

عمل جميل @ a13m!
في غضون ذلك: https://developer.github.com/changes/2017-09-06-protected-branches-preview-end/.

يبدو أننا عالقون حتى يقوم شخص ما بحشد الطاقة لمعالجة هذا الأمر.

أنت الآن مطالب بتمرير الكائن required_pull_request_reviews عند استدعاء نقطة نهاية حماية فرع التحديث. كان تمرير هذا الكائن اختياريًا أثناء فترة المعاينة.

tomgoren في ٣ أكتوبر ٢٠١٧

انا احب ان ارى هذا ايضا العلاقات العامة من فضلك ❤️

sfdye في ٦ أبريل ٢٠١٨

s-t-e-v-e-n-k في ٣٠ أبريل ٢٠١٨

👍3

مرحبًا يا رفاق ، لم يكتمل التصحيح الخاص بي حتى الآن ، لكنني اعتقدت أنه سيكون من الجيد الحصول على ملاحظات مبكرة ^

JPWKU في ١٠ مايو ٢٠١٨

لقد كنت أعمل أيضًا على هذا ، وأتناوله بطريقة مختلفة - عن طريق إضافة التوابع إلى الفرع ، وإسقاط Repository.protect_branch ()

s-t-e-v-e-n-k في ١١ مايو ٢٠١٨

stevenk آه أرى ذلك الآن. حسنًا ، لا تريد تكرار العمل. كنت أنوي الحفاظ على مكتبة API كما هي. كيف تريد أن تفعل ذلك؟

JPWKU في ١١ مايو ٢٠١٨

JPWKU أنا آسف ، كان يجب أن أبلغ بشكل أكثر وضوحًا أنني كنت أعمل بنشاط على هذا. تتطابق طريقة Repository.protect_branch () مع واجهة برمجة تطبيقات GitHub ، حتى قاموا بإزالتها عند تخرج واجهة برمجة تطبيقات حماية الفرع الجديدة.

يقوم التصحيح الحالي الخاص بي بتنفيذ واجهة برمجة تطبيقات حماية الفروع كما يظهر GitHub على https://developer.github.com/v3/repos/branches/ ، ونتيجة لذلك ، فهو كبير نوعًا ما. لدي 3 حالات اختبار أو نحو ذلك لإنهاء الكتابة قبل أن أرفع هذا الأمر.

s-t-e-v-e-n-k في ١٢ مايو ٢٠١٨

stevenk لا تقلق ، نظرًا لأنك قد حصلت بالفعل على عمل في هذا المجال ، فسوف أقوم بإغلاق العلاقات العامة الخاصة بي والتعليق / المساعدة بشأنك.

JPWKU في ١٢ مايو ٢٠١٨

stevenk هل تخطط لدمج هذا قريبًا؟ أنا في حاجة إلى هذه الوظيفة.