Код-ревью

• Не стесняйтесь обращаться к рецензентам, отправляя им URL вашего пул-реквеста через мгновенные сообщения или электронную почту. Делайте это после того, как пул-реквест пройдёт CI.

• Отвечайте на каждый комментарий рецензентов. Если вы согласны с комментарием, напишите «Done», если нет — объясните причину.

• Чтобы не перегружать рецензентов уведомлениями по электронной почте, вы можете отвечать на их комментарии пакетом.

• Сократите количество ненужных коммитов. Некоторые разработчики делают коммиты часто. Рекомендуется объединять последовательность небольших изменений в один коммит, используя команду git commit --amend вместо git commit.

Стандарты кодирования

Стиль кода

Наш код на C/C++ следует руководству по стилю Google.

Наш код на Python следует PEP8.

Наша система сборки помогает проверять стиль кода.

Модульные тесты

Пожалуйста, не забывайте добавлять соответствующие модульные тесты в качестве части своего вклада.

• Для кода на C/C++, пожалуйста, следуйте Primer для google-test.

• Для кода на Python, пожалуйста, используйте стандартный пакет Python unittest.

Отчётность об уязвимостях (проблемы безопасности)

Если вы обнаружите значительную уязвимость или её признаки, отправьте электронное письмо на адреса, указанные ниже. Мы сообщим вам о следующих шагах. В настоящее время контакты по вопросам безопасности:

• DongDaxiang dongdaxiang@baidu.com;
• GongWeibao gongweibao@baidu.com;
• WangXi wangxi16@baidu.com.

При отправке сообщений об уязвимости используйте систему электронной почты (например, Gmail), поддерживающую шифрование между узлами с использованием STARTTLS. Примеры таких систем включают Gmail, Outlook.com и runbox.com. Подробнее об усилиях по поощрению использования STARTTLS см. в статье STARTTLS Everywhere. Ваш почтовый клиент должен использовать шифрование для связи с вашей почтовой системой (то есть, если вы используете веб-клиент электронной почты, то используйте HTTPS, а если вы используете программное обеспечение почтового клиента, то настройте его на использование шифрования). Шифрование между узлами не так надёжно, как сквозное шифрование, но мы решили, что оно достаточно надёжно для этой цели, и гораздо проще заставить всех его использовать.

Мы будем рады отдать должное всем, кто сообщает об уязвимости, чтобы мы могли её исправить. Если вы хотите остаться анонимным или псевдонимным, сообщите нам об этом; мы с радостью учтём ваши пожелания.