Ударим качеством по качеству

[cjelli]

У нас начальство вдруг решило, что качество кода никуда не годится, и начало борьбу за его, качество, повышение. Была сформирована группа, которая ведает шлагбаумами на merge requests, только результат пока не очень. Нас тоже попросили помочь - и в первую очередь бросили на самую неквалифицированную работу, разгребать, почему тесты валятся, но теперь вот дошли руки посмотреть на код шлагбаумистов:

def is_checkin_approved(labels):
    res = False
    res |= isLabelPresent(labels, 'checkin::approved')

    if not res:
        logging.info(f"Checkin is not approved for the MR."
                     f"Refer https://company.internal.site.com/Main+Controlled+Check-in+Guidelines")
    return res

def isLabelPresent(labelscol, label):
    exist_count = labelscol.count(label)
    return exist_count > 0

Здесь прекрасно все, но захотелось лучше понять, с кем мы имеем дело. Я спросил одного из них, уверен ли он, что bitwise or - это то, что ему нужно.

Ответ был: ачотакова, все работает.
 

 

Эти люди "ответственны" за повышение качества кода.

Эти люди - тот самый продукт южноазиатского конвейера, выпускающего 30 тыс. "инженеров-программистов" в год.

Comments

[juan_gandhi]

С южной азией действительно тяжело. У них там принято чтить карго-культ.

А чо, лейбл нельзя просто на гитхабе зафигачить? (Just asking.)

На днях вставил в мой скрипт curl на github api, чтобы вылавливать нужные мне метки (а по ним я уже генерю всякое).

[cjelli]

Это часть, эссенция скрипта в сто строк. И это не гитхаб.

[juan_gandhi]

Oh, ok.

[sobriquet9]

Если до одной строчки сократить, то меньше строк кода получится. Меньше строчек - меньше продуктивность.

А ещё чтоб было куда breakpoint поставить.

[e2pii1]

Мерить продуктивность числом строк кода - это как мерить продуктивность официанта числом разбитых тарелок