search

xDrivenDevelopment / precommit1c

Набор утилит для автоматической разборки/сборки внешних обработок и отчетов, при помещении (commit) в git
Other
224 stars 62 forks source link

Показательный pull request для любителей вебинаров #87

Closed ghost closed 7 years ago

ghost commented 8 years ago
zeratulayuris commented 8 years ago

Во-первых, покаЗАтельный (или покаРАтельный :) PR. Во-вторых, mklink не скотина, а вполне себе утилитка.

artbear commented 8 years ago

Хорошая идея. В будущем хорошо бы доработать для передачи пути к репозитарию через командную строку и выполнения полностью автоматических шагов.

ghost commented 8 years ago

Коллеги. Может все таки - если вам интересен pull request поучаствуете кодом а не комментариями.

Ваши коменты отталкивают контрибьютора ;-), о чем я сегодня еще раз скажу на вебинаре. Все такие умные, аж страшно.

Pull request'ы имеют возможность дорабатываться коллективно, а не только обсуждаться

artbear commented 8 years ago

Жду исправлений по замечаниям, хотя бы самые важные, тогда смогу принять PR

artbear commented 8 years ago

ОФФ/2

Коллеги. Может все таки - если вам интересен pull request поучаствуете кодом а не комментариями. Ваши коменты отталкивают контрибьютора ;-), о чем я сегодня еще раз скажу на вебинаре. Все такие умные, аж страшно. Pull request'ы имеют возможность дорабатываться коллективно, а не только обсуждаться

От контрибьютора не требуется многого, всего лишь завершить свою работу, слегка отшлифовать и избавиться от проблемных/ошибочных участков кода. Зачем принимать "грязный" или незавершенный код, когда можно его слегка доработать до чистоты? Контрибьютор часто больше в контексте, чем ревьюеры, и ему легко поправить большую часть замечаний. А вообще контрибьюторов мы уважаем, ценим и помогаем :)

artbear commented 8 years ago

Как я уже писал выше, конечно, этот PR интересен.

ghost commented 8 years ago

@artbear

Pull request'ы имеют возможность дорабатываться коллективно, а не только обсуждаться

ghost commented 8 years ago

@artbear @EvilBeaver @zeratulayuris вынужден пояснить методологически

поясню:

  1. после работы с командой-клиента на ретроспективе нам было сказано "неудобно устанавливать precommit1C"
  2. я имел неосторожность сказать ребятам команды клиента "Ждем вашего пулреквеста" - так как собственно частично в обучение входила работа с OpenSource проектами в режиме "НЕ только чтение"
  3. команда заявила что "К вам на хромой кобыле не приедешь - слишком вы крутые"
  4. после открытого вебинара мы сделали в режиме парного программирования такой вот скрипт на Oscript - очень полезно при обучении работы с библиотеками и синтаксис помощником
  5. естественно сделали показательный pull request
  6. скрипты костыльные и не учитывают все возможные варианты, например под windows 7 mklink не сработает (требуется UAC), копирование v8unpack подразумевает нашу установку (мы рекомендуем ставить OScript как и Python по адресу c:\Oscript
  7. но скрипты у контрибьютора позволяют решить "текущую боль" - поэтому он здесь ими делится

В итоге - команда которую мы консультировали оказалась отчасти права: "не подъедешь на хромой кобыле" - заплюют ;-)".

pull request - позволяет вести коллективную доработку pull request'а, без необходимости тратить время на code review и комментарии

При текущем подходе мы только оттолкнем обычных 1С специалистов от контрибьютинга, так как ждать от них уникальных скриптов пока рано.

Я знаю что @nixel2007 планирует инсталятор через opm - его даже обсуждали, но боль с первой установкой precommit1C есть уже сейчас.

Текущий Pull request можно отменить - ;-), чтобы не холиварить.

artbear commented 8 years ago

@allustin обсуждать по схеме работы пулл-реквеста (PR) предлагаю в рамках гиттера проекта

PR зря закрыл, он интересен + ты же хотел дождаться контрибьюторов, которые за тебя доделают работу (перевожу "выполнят коллективную доработку pr") Тогда PR должен быть открытым, иначе его никто не найдет.

artbear commented 8 years ago

@allustin Отписался в гиттере

nixel2007 commented 7 years ago

Грустно и печально.

nixel2007 commented 7 years ago

Закрыл, так как инсталлятор реализован через --install.