ANDROID_LIVE Telegram 267
​​Pull request. Как правильно организовать?
#разработка #вопрос #комментарии

Во всех компаниях, которые так или иначе связаны с разработкой, используется система контроля версий. И одна из самых распространенных является Git.

Для того, чтобы обеспечить надлежащее качество кода, используется pull request. По своей сути, это возможность другим участникам репозитория посмотреть изменения в коде, увидеть потенциальные или явные проблемы в нем и сообщить об этом тому, кто этот код писал.

Этот процесс позволяет другим участникам проекта ознакамливаться с новым кодом, предотвращать потенциальные баги и держать кодовую базу в общем ключе. Единственным минусом являются временные затраты.

Мы всем используем эту технику, но насколько правильно? Например, в нашей команде существует ряд правил, которые связаны с PR:
1) Объем каждого PR не должен превышать 500 строк кода. Если же фича занимает больший объем, то ее надо разбить на несколько PR, помечая, что текущая ветка должна вливаться не в dev, а в родительскую.
2) Каждый PR должен сопровождаться кратким описанием того, что в нем было сделано.
3) Добавление ресурсов и переименование желательно выносить в отдельный PR.
4) Те pull request, которые надо влить в общую ветку должны быть проверены при помощи тестов, развернутых на CI.

Эти правила помогают нам лучше организовывать работу в команде, связанную с созданием PR.

А какие есть правила у вас в команде, связанные с PR?
Очень интересно узнать и взять на заметку что-то новое для себя. Прошу поделиться в комментариях.
👍1



tgoop.com/android_live/267
Create:
Last Update:

​​Pull request. Как правильно организовать?
#разработка #вопрос #комментарии

Во всех компаниях, которые так или иначе связаны с разработкой, используется система контроля версий. И одна из самых распространенных является Git.

Для того, чтобы обеспечить надлежащее качество кода, используется pull request. По своей сути, это возможность другим участникам репозитория посмотреть изменения в коде, увидеть потенциальные или явные проблемы в нем и сообщить об этом тому, кто этот код писал.

Этот процесс позволяет другим участникам проекта ознакамливаться с новым кодом, предотвращать потенциальные баги и держать кодовую базу в общем ключе. Единственным минусом являются временные затраты.

Мы всем используем эту технику, но насколько правильно? Например, в нашей команде существует ряд правил, которые связаны с PR:
1) Объем каждого PR не должен превышать 500 строк кода. Если же фича занимает больший объем, то ее надо разбить на несколько PR, помечая, что текущая ветка должна вливаться не в dev, а в родительскую.
2) Каждый PR должен сопровождаться кратким описанием того, что в нем было сделано.
3) Добавление ресурсов и переименование желательно выносить в отдельный PR.
4) Те pull request, которые надо влить в общую ветку должны быть проверены при помощи тестов, развернутых на CI.

Эти правила помогают нам лучше организовывать работу в команде, связанную с созданием PR.

А какие есть правила у вас в команде, связанные с PR?
Очень интересно узнать и взять на заметку что-то новое для себя. Прошу поделиться в комментариях.

BY Android Live 🤖




Share with your friend now:
tgoop.com/android_live/267

View MORE
Open in Telegram


Telegram News

Date: |

Among the requests, the Brazilian electoral Court wanted to know if they could obtain data on the origins of malicious content posted on the platform. According to the TSE, this would enable the authorities to track false content and identify the user responsible for publishing it in the first place. During the meeting with TSE Minister Edson Fachin, Perekopsky also mentioned the TSE channel on the platform as one of the firm's key success stories. Launched as part of the company's commitments to tackle the spread of fake news in Brazil, the verified channel has attracted more than 184,000 members in less than a month. 6How to manage your Telegram channel? SUCK Channel Telegram “Hey degen, are you stressed? Just let it all out,” he wrote, along with a link to join the group.
from us


Telegram Android Live 🤖
FROM American