Должно ли быть стыдно?

почему не thunderbird?

Ты хочешь, чтобы для тебя сделали Code Review? Если да, так и напиши, а то сразу не понятно.
А вообще - отправляй, примут - ништяк, не примут - допилишь и ещё раз отправишь. На качество кода всем плевать.

почему не thunderbird?

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

На качество кода всем плевать.

Вообще код в Trojita очень даже. И архитектура нормальная.

Если да, так и напиши, а то сразу не понятно.

fixed

Лично мне в Trojita не понравились две вещи. Во-первых, поддержка только одного ящика (если там такая чудная архитектура, почему ещё не исправили?), и во-вторых дизайн, мне все-таки ближе что-то в духе Apple Mail.

https://loki13@bitbucket.org/loki13/trojita.git

Забавно, сафари пишет «Possible Phishing Site».

https://loki13@bitbucket.org

Пароль забыл указать

Да пришли им патч, че. Не понравится — не примут. Если что, поправят. Не попробуешь — не узнаешь

Пароль забыл указать

Зачем для git clone пароль? Ну или из своего битбакета зайти в репу.

Upd^ Понял. Отредактировал. Хотя при заходе по той ссылке юзернэйм сам пропадает, если куков нет.

Спасибо за комментарий, а то я после прочтения поста решил попробовать вместо громоптицы, но после

поддержка только одного ящика

передумал.

Тебя никто не забанит и не начнёт травить за плохой стиль кода в патче. Просто в крайнем случае укажут, что нужно поправить. А может и сами поправят. В любом случае у тебя есть много попыток прислать пулл-реквест. И если рабзарботчики в принципе не против данных изменений и ты улучшаешь код с каждой попыткой, то никто не обидится на то, что ты смог добиться нормального качества лишь с 10 попытки. Но с другой стороны это не ядро Linux и требования и занятость разработчиков по-любому слабее. Так что я ставлю на то, что они примут как есть или сами исправят все недочёты.

Почитай это (flaska.net) если еще не, и шли патчи. Написано ж - контрибьюторам велкам.

Да, шлите в апстрим обязательно! Есть шанс, что вас обложат матом по первому разу, но все равно надо.

Попробуй Geary, вполне неплохой клиентик.

/me делает код-ревью каждый день на регулярной основе.

Почти никогда не было реакции в виде массивного фейспалма и никогда такой реакции не видел. Если код ревьюят, то он хороший, даже если его ругают. Хуже, если совсем не ревьюят.

Должно ли быть стыдно

Должно. Всю жизнь следует стыдится. Это такая христианская добродетель.

стоит ли попытаться эти патчи послать в апстрим или мне должно быть стыдно и лучше сидеть в тёмном углу

Не «или», а «и». Попытаться послать нужно, а также нужно и стыдится сидя в своей кельи.

мне должно быть стыдно и лучше сидеть в тёмном углу и патчить этим только свою локальную версию?

Отдавать что-то на code review, не нак страшно как кажется. Как подсказывает практика, ты всегда будешь думать что твой код тупее, чем он есть на самом деле.

Комплекс быдлокодинга не есть хорошо.

Комплекс быдлокодинга не есть хорошо.

Да у меня особо комплекса нет. Просто есть такие вещи как стиль кода, определенные правила. А я пишу как сам умею. В принципе умею нормально, 20 лет уже опыта скоро будет(Хотя в кутях всего год, да).

20 лет уже опыта скоро будет

Тогда чего стесняться?

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

стоит ли попытаться эти патчи послать в апстрим

да как хочешь в общем то т.к. кроме тебя и девелов проэкта всем по барабану.

да как хочешь в общем то т.к. кроме тебя и девелов проэкта всем по барабану.

Я вообще когда создавал тред, думал что хоть один человек коммиты глазами поглядит и скажет что. Никто даже не смотрел, по крайней мере про конкретный код ни одного слова в комментах. Наверное надо было в девелопмент, а не в толксах создавать.