یه نفر باگم رو درست کرد، بعد فهمیدم نیمه اشتباهش رو درست کرده، ولی بازم کار کرد!
یک توسعهدهنده ناشناس پولریکوئستی روی سرور MCP سافاری فرستاد که سه تابع رو با هم رفت و یه باگ واقعی رو فیکس کرد. نویسنده وقتی نشست کد رو دقیقتر بررسی کنه، فهمید نصف پچ هیچ کاری نمیکنه — نه یه اثر ظریف، واقعاً هیچ. ولی با این حال، باگ حل شده بود.
چرا مهمه؟
این ماجرا نشون میده حتی وقتی کد کار میکنه، لزوماً فهم درستی از اون نداریم. توسعهدهندهای که فقط CI سبز رو میبینه و merge میکنه، ممکنه کد مرده رو هم با خودش بیاره توی پروژه. این اتفاق دقیقاً نشوندهندهی اهمیت درک عمقی کد، حتی وقتی ظاهراً مشکلی حل شده.
به درد کی میخوره؟
توسعهدهندگان نرمافزار مرورگرهای کد (Code Reviewerها) مدیران تیمهای فنی کسانی که با MCP یا پروژههای متنباز کار میکنن
تو عمل چی کار کنیم؟
دفعه بعدی که یه پولریکوئست CI سبز داشت و باگ رو فیکس کرد، وایستید و دقیق بخونیدش. ببینید هر خط کد واقعاً داره چیکار میکنه، نه فقط اینکه تستها رد شدن. کد مرده یه روز ممکنه با تغییر شرایط فعال بشه و دردسرساز شه.
نظر Blue IT News
CI سبز تضمین کیفیت کد نیست، فقط تضمین اجرای تستهاست. هر پچی رو با چشم خودتون بخونید، حتی اگه از یه توسعهدهنده معتبر اومده باشه.