Gitwatch: إضافة خيار لتضمين فرق (مقطوع) في رسالة التنفيذ

يمكنك تنفيذ مفتاح خيار (على سبيل المثال -d / - diff) لتضمين ذلك ، ومفتاح إضافي لتعيين بعد عدد الأحرف التي يجب اقتطاعها (يجب أن يكون افتراضيًا إلى 80)

هذا يبدو وكأنه طريقة معقولة للقيام بذلك.

لقد تحققت مؤخرًا من عدم وجود طريقة _good_ للقيام بمفاتيح خيارات طويلة (- diff) في bash (هناك بعض الطرق المعقولة ، لا أعتبر أيًا منها أنيقًا ولكن نادراً ما تكون برمجة bash أنيقة ؛)) نظرًا لأن هذا هو bash ، فمن المحتمل أن أختار getopts وأقوم فقط بالتبديل بحرف واحد على الرغم من وجود طريقة للقيام بخيارات طويلة تعتمد على سلوك غير قياسي أو شيء من هذا القبيل. هناك أيضًا GNU getopt الذي يدعم الخيارات الطويلة ولكن قد يكون حلًا أقل قابلية للحمل ويبدو أقبح قليلاً. على أي حال ، ربما لا تهتم بكل ذلك في هذه المرحلة لأن gitwatch حتى الآن لا تتخذ أي خيارات بخلاف المسار ، لكن أعتقد أن الأمر متروك لك فيما تعتقده بشأن ذلك. لقد عملت مع getopts من قبل ، إنه أمر بسيط إلى حد ما وهو عبارة عن bash مدمج.

أعتقد أن السؤال الآخر هو كيفية الاقتطاع وما إذا كنت تريد تضمين رموز + وما إذا كنت تريد أسماء الملفات أيضًا. ربما يمكن أن يكون هناك عدد قليل من الخيارات التي تضيف مكونات مختلفة للرسالة (فرق مقطوعة ، ملفات متغيرة ، رسالة الالتزام التلقائي الحالية ، إلخ).

قدمنا ​​بعض الخيارات القصيرة في فرع الاختبار ، باستخدام getopts في bash. لكنك على حق ، إذا لم تكن هناك طريقة "جيدة" لاستخدام الخيارات الطويلة ، فإن الخيارات القصيرة ستفعل في الوقت الحالي.

في الوقت الحالي ، ما عليك سوى إضافة رسالة diff قبل أن تنجح بقية رسالة الالتزام. يمكن للمستخدمين تعديل البرنامج النصي بسهولة كافية إذا كانوا يفضلون نوعًا مختلفًا من رسالة الالتزام. إذا تمكنت من سحب فرع الاختبار وإضافة الخيار والتعامل معه ، فسيكون ذلك رائعًا: د

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

أنا في الواقع أبحث عن شيء مشابه ، لكنني أريد فقط تضمين اسم الملف في رسالة الالتزام. ربما يكون هذا أسهل أو قد يكون هناك بالفعل طريقة للقيام بذلك؟

dmusican أريد أيضًا هذه الميزة ، لذلك قمت بتنفيذها وأنشأت طلب سحب مقابل أحدث فرع رئيسي لك لها: https://github.com/gitwatch/gitwatch/pull/54
أيضًا ، نظرًا لأنه يمكنك تعيين عدد الأسطر التي يتم عرضها كـ "git diff" مقابل "git diff --stat" ، فإن استخدام "-l 1" سيؤدي إلى الوظيفة التي حاول شخص ما تنفيذها في طلب سحب قديم https: // github.com/gitwatch/gitwatch/pull/23/commits/af28cf4a87c71cdb9da7046d40ea9542a4661f43

mlncn إذا حاولت طلب السحب https://github.com/gitwatch/gitwatch/pull/54 ( git fetch origin pull/54/head:pr-54 && git checkout pr-54 ) ، يجب أن تكون قادرًا على رؤية أسماء الملفات (مع رسم بياني للتغييرات) على أنها الالتزام بالرسالة باستخدام gitwatch -l 1 ~/myproject .

تم دمج طلب السحب. شكرا جزيلا ، shervinemami .