Как бороться с доброжелательными участниками с открытым исходным кодом?

30

Если программист связывается с вами и просит внести свой вклад в ваш проект, как вы справляетесь с этим?

  • Ты не знаешь, хорош ли этот парень. Возможно, у него будет больше проблем, чем он того стоит.
  • Возможно, он пытается прикрепить свое имя к успешному проекту только за похвалы.
  • Возможно, он пытается продвинуть проект в направлении, которое вам не нужно, добавляя функции, которые, по вашему мнению, не стоят дополнительной сложности.
  • Или он может быть очень полезным автором. Ты просто не знаешь.

Как вы обрабатываете такие запросы от людей, которых вы не знаете (в частности, на GitHub, если это имеет какое-то значение)? Какой здесь этикет?

Соло
источник
44
на github просто позвольте ему сделать запрос на удаление и отклонить, если вам не нравится
трещотка урод
11
Подходящее имя учетной записи?
Арен
5
Я думаю, что вы, возможно, имеете в виду статус активного участника. Короче говоря: не делайте кого-то вкладчиком в то, чему он еще не способствовал. Преимущество проектов с открытым исходным кодом заключается в том, что каждый может над этим поработать, поэтому пусть они это сделают, а затем решат.
Ткните
2
@ Разве я могу получить +=1за это дополнительную плату ?
Соло

Ответы:

41

Членство в проекте OSS - это не то же самое, что финансируемая корпоративная команда, в которой люди проходят собеседование и выбираются. Исходный код уже есть (иначе это не открытый код). Скажите им, чтобы отправить в некоторые патчи. Если это хорошие исправления (и вы должны сначала просмотреть их), зафиксируйте их. Как только потенциальный клиент укрепит доверие и историю внесения ценных вкладов, предоставьте ему доступ для записи.

Команды OSS растут органично. Сделайте себя доступными, чтобы задавать вопросы, и пусть они со временем приобретут уличный авторитет.

Майкл
источник
+1 это. Патчи - лучший способ начать, потому что они создают самый низкий начальный барьер для запуска. Если человек начинает предоставлять больше патчей (или становится постоянным автором), то быстрее / проще переключиться на получение запросов.
Эван Плейс
Извините за мое невежество, но есть ли разница между патчем и запросом на извлечение?
heltonbiker
54

Почему бы не позволить этому нетерпеливому человеку отправить вам запрос на удаление? У вас будет возможность просмотреть и критиковать код этого человека. Это кажется самым простым решением.

duggiefresh
источник
34
Разве так GitHub не должен работать?
Alroc
14
Единственное, что я хотел бы добавить, - попросить этого человека, чтобы первый пул-запрос был чем-то маленьким, чтобы он не потратил слишком много усилий, прежде чем вы сможете увидеть его код. Столкнувшись с подобным «испытанием», они могут почувствовать себя обязанным представить что-то большое и впечатляющее :-)
Кшиштоф Козельчик
1
и что-то большое и впечатляющее сломает вещи, если не обращаться с ними осторожно
чокнутый урод
Итак, если вклад полезен, но ему не хватает качества, и вы слишком заняты, чтобы держать его за руку, можно ли просто отклонить вклад?
Соло
@ Соло: Конечно. Также можно ожидать / просить участника поддерживать и исправлять ошибки в их функции, по крайней мере, до тех пор, пока остальная часть команды не изучит этот код, особенно для больших коммитов.
Хьюго
16

Принимаю и проверяю изменения от него на временной основе. Дайте ему доступ для записи в хранилище исходного кода, когда он доказал свою ценность.

Роберт Харви
источник