web-standards-ru / podcast-old

Выпуски подкаста «Веб-стандарты»
Other
233 stars 22 forks source link

Обновит и упростит CONTRIBUTING.md #149

Closed firefoxic closed 7 years ago

firefoxic commented 7 years ago

Участие понятнее стало:

Набор — проще некуда. Редактирование — с учётом яспеллера. Форматирование с построчной разбивкой и html-комментарием для тудушек (так и заметнее в редакторе, и — самое главное — в скомпилированном варианте не видно читателям, если вдруг пришлось принять недовершённый пулреквест). Думал больше выкинуть, но не получается пока.

firefoxic commented 7 years ago

Правил отдельно (в лаверне вообще) и в творческих муках. Поэтому одним коммитом с бессмысленным заголовком и тонной пояснений :(

firefoxic commented 7 years ago

У меня ещё одна идея есть! Для этого тоже нужны будут правки в контриб — ПР пока не принимать ;)

firefoxic commented 7 years ago

Так вот. Идея в том, чтобы отказаться от отдельной ишьи для каждой темы. Слишком овер-дофига их с такими темпами расшифровки. Для каждого выпуска одна ишья. В ней в виде туду-списка — заголовки тем с таймингом. Кому, что нравится тот, то и набирает, просто отписывается об этом. Набраные отмечаем галкой. Занятые помечаем тоже как нибудь (по ходу придумаю, может ником через собаку, или ссылкой на комментарий соответстующий). В пулреквестах не пишем Close/Closed/и прочие закрывающие ишью слова, а просто номер через решётку — с ишьей появляется связь, но принятие ПР не приводит к закрытию ишьи. Получается, что-то вроде такого: screenshot_20161224_184607 Вроде становится меньше заморочек с выбором для набора, с названиями веток. Ну и в ишьях порядок наведётся (после мержа ПР займусь переименовыванием).

Jazzis18 commented 7 years ago

@firefoxic Ишьюс действительно очень много выходит, можно запутаться или совершать лишние действия, чтобы понять какие темы уже взяты, а какие свободны (на самом деле не так много действий :smile:).

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

Jazzis18 commented 7 years ago

@firefoxic Хочу попробовать сделать набор темы и ПР по твоему контрибу. Для этого, как я понял, должна уже существовать ишья для набираемого выпуска (я пилю События 31-го выпуска). Я уже завёл отдельную ветку для выпуска 31 и положил свои изменения.

Вижу уже есть ишьи для 1 по 23 выпуски, а далее топает для каждой темы выпуска своя ишья. Ты как будешь делать "внедрение" — создашь ишьи по новой (для тех выпусков, которые разделены) или же переименуешь какие-то по типу "Выпуск №XX", а остальные закроешь?

P. S. Хотя, у меня есть и отдельная ишья для темы выпуска. Для нынешней версии контриба, так сказать.

firefoxic commented 7 years ago

@Jazzis18 переименую, добавлю темы, затем лишнее закрою. Пока ПР не отправляй. Подожди пока ишьи причешу.

firefoxic commented 7 years ago

Ещё один профит нашёл — метка В работе будет не нужна совсем. Можно будет удалить её. И это хорошо, ибо доставало немного вечно её ставить-убирать, ставить-убирать.

firefoxic commented 7 years ago

Порядок в ишьях наведён! Ещё одним профитом получилось то, что теперь в списке сразу видно, в каких выпусках сколько тем набрано. Можно сориентироваться по этим прогесс-барам и добивать уже начатые, а не гулять по всем выпускам набирая по одной теме. Ну или наоборот, если хочется нетронутый взять целиком — смотрим на пустые.

P.S. Ишьи есть аж до 65го выпуска %) Нужно просто из списка закрытых по необходимости заново открывать и добавлять туду-список тем.

pepelsbey commented 7 years ago

Я понимаю, что это всё может звучать по-свински, но скажу.

Меня очень смущает каждое предложение с отдельной строчки: читать сгенерированный из этого HTML легко, но если каким-нибудь редактором открыть такой файл (а я часто так делаю), то читать становится невозможно — как поэма какая-то. Привычный поток текста начинает рваться на псевдо-строфы или абзацы.

Надо наверное как-то разделить процесс набора (и все удобства, с ним связанные) от результата.

firefoxic commented 7 years ago

Не, процесс набора тут не при чём. Набирать что так, что эдак — одинаково (хотя может и удобнее не успел толком понять). А вот diff в коммитах, пулреквестах и так далее, с длиннющими строчками мне немного напоминал боль автора этой статьи. То есть это скорее про процесс редактирования, а не набора.

Но если читаемость важнее и она хромает — вернуть назад думаю не составит труда. Но уже не сегодня — глаза вытекают уже с этими ишьями %)

firefoxic commented 7 years ago

может звучать по-свински

Вроде не хрюкал ;)

pepelsbey commented 7 years ago

На самом деле, есть удобный режим инлайнового диффа для текстов, не для кода.

pepelsbey commented 7 years ago

Он переключается здесь

image

firefoxic commented 7 years ago

Завтра, всё завтра :)

Но вот имя автора реплики всё же лучше на отдельной строке оставить — так и в текстовом варианте считывается легче, и набирать удобнее, вернее копипастить эту короткую строчку.

Это оставить?

firefoxic commented 7 years ago

И ещё, @pepelsbey, что по ишьям скажешь? Не перегнул я палку?

Jazzis18 commented 7 years ago

@firefoxic Имхо, надо оставлять Assignee, чтобы было проще найти какими выпусками ты занят. Ишью стало меньше, и это немножко упростило поиск из-за того, что по кол-ву TODO теперь можно понять, какой выпуск уже начат, а какой — нет. Но в ситуации, когда человек забросил, а потом решил вернуться (как я, например) будет ещё проще найти где я занят по своей аватарке справа от ишьи :D

Следование старому контрибу, когда для каждой темы была своя ишья и ветка именовалась по типу issues/<номер ишьи темы> конечно может помочь в некоторых случаях, но не во всех.

firefoxic commented 7 years ago

@Jazzis18 дело в том, что в Assignee можно поставить только тех, кому Вадим дал те права, что у меня с тобой и ещё у пары-тройки человек есть. А раздавать их вообще любому, кто просто в ишье отписался один раз когда-то (а потом может и забыл об этом) — это как-то не очень. Я был удивлён когда Вадим мне их дал. Хоть я к тому времени и сделал уже что-то, а не просто отписался, мне тогда казалось, что вроде как возможности ишьи заводить и пулреквесты отправлять вполне хватает. Но сейчас мне эти права дали возможность разрулить висячие пулреквесты, не дёргая Вадима в субботу в день записи подкаста.

Так что по факту Assignee бесполезно. Равно как и метка «В работе». @pepelsbey можно её наверно либо удалить либо переименовать во что-то более полезное. Я просто, когда последнюю такую метку убрал, не нашёл где они редактируются.

firefoxic commented 7 years ago

А искать ишьи где отписался лучше всего поиском через поле фильтров, например так. Но нам с тобой это не подходит, у нас жутко длинный список :)

Jazzis18 commented 7 years ago

@firefoxic Редактировать метки можно через кнопку "Labels", которая справа от фильтра. image По поводу Assignee понял. Так и думал, что что-то упускаю.

firefoxic commented 7 years ago

Когда-нибудь я его допилю 😅

firefoxic commented 7 years ago

Хватит ему валяться :)