tag:blogger.com,1999:blog-6346131298481535631.post3855462102106875181..comments2023-05-18T15:33:51.966+03:00Comments on Чудес не бывает или я ошибаюсь?: Code review в Visual Studio 2012 - часть 2Maxim Shulga (aka MaxBeard12)http://www.blogger.com/profile/05615743910272666556noreply@blogger.comBlogger18125tag:blogger.com,1999:blog-6346131298481535631.post-42542491784927545252014-06-25T17:12:27.303+04:002014-06-25T17:12:27.303+04:00И да, можно комментить прямо в коде, через веб: зн...И да, можно комментить прямо в коде, через веб: знай и люби свой инструмент. Плохо, что оповещения не приходят :(Максим Шульга (Maxim Shulga)http://maxshulga-ru.blogspot.com/noreply@blogger.comtag:blogger.com,1999:blog-6346131298481535631.post-2580040987373111452014-04-14T08:47:17.571+04:002014-04-14T08:47:17.571+04:00Спасибо за комментарий :) А под re-review понимает...Спасибо за комментарий :) А под re-review понимается трекинг того, как был изменен код по комментариям из первого ревью? Если так, то да - с этим проблемы. Получается, что пока нужно заводить следующее ревью, на след chanset или shelve. Но это неудобно: нет связи с предыдущим. Основная проблема ревью в студии+TFS, это невозможность просматривать изменения между пачками changeset'ов. Это бы решило проблему, хотя и частично. Review Assistant это умеет. Общался с товарищами из MS на эту тему, обещали, что в след версии будут улучшения code review в студии. Но пока публичных новостей на эту тему не слышал.Максим Шульга (Maxim Shulga)http://maxshulga-ru.blogspot.com/noreply@blogger.comtag:blogger.com,1999:blog-6346131298481535631.post-48627510615573742142014-04-11T18:18:24.169+04:002014-04-11T18:18:24.169+04:00Спасибо что поделились опытом. Мы сейчас тоже эксп...Спасибо что поделились опытом. Мы сейчас тоже эксперементируем. Пока есть трудности с подходом re-review.Igor Skomorokhnoreply@blogger.comtag:blogger.com,1999:blog-6346131298481535631.post-49189042300449720662013-11-14T23:11:25.717+04:002013-11-14T23:11:25.717+04:00Да не столько ты зацепил, тебе просто досталось. :...Да не столько ты зацепил, тебе просто досталось. :) Просто у нас тоже есть адепты "единых стандартов", в стиле "да пусть херово выглядит и работает зато все в одном месте" . На счет написать, то че то я обленился (могу попробовать спрятаться за фразой "много работы") ,но мысли написать про то что я юзаю у меня давно есть.Maxim Shulga (aka MaxBeard12)https://www.blogger.com/profile/05615743910272666556noreply@blogger.comtag:blogger.com,1999:blog-6346131298481535631.post-60693893007456055072013-11-14T20:26:53.931+04:002013-11-14T20:26:53.931+04:00:) я искренне рад, что тебя эта тема зацепила. Я п...:) я искренне рад, что тебя эта тема зацепила. Я постараюсь посмотреть то, что ты рекомендуешь. Но если ты сделаешь подобный обзор по своим тулам - будет супер. Очень поможет :)Maxim Shulga (aka MaxBeard12)https://www.blogger.com/profile/05615743910272666556noreply@blogger.comtag:blogger.com,1999:blog-6346131298481535631.post-62398716770076061312013-11-14T19:25:28.731+04:002013-11-14T19:25:28.731+04:00Макс, исходники они у любых продуктов исходники. Н...Макс, исходники они у любых продуктов исходники. Никто не предлагает хранить там приватные ключи для подписей. Их и в локальном TFS хранить не следует.Maxim Shulga (aka MaxBeard12)https://www.blogger.com/profile/05615743910272666556noreply@blogger.comtag:blogger.com,1999:blog-6346131298481535631.post-57749284885245551542013-11-14T18:26:06.183+04:002013-11-14T18:26:06.183+04:00Попробовать github то я могу, дальше то что? Надо ...Попробовать github то я могу, дальше то что? Надо представлять плюсы/минусы всех решений. Для этого надо пробовать и TFS тоже. А насчет хранения, даже платного, исходников продуктов по безопасности в облаках - это (пока) выше моего понимания :)Maxim Shulga (aka MaxBeard12)https://www.blogger.com/profile/05615743910272666556noreply@blogger.comtag:blogger.com,1999:blog-6346131298481535631.post-33846026656515942972013-11-14T18:13:04.360+04:002013-11-14T18:13:04.360+04:00Макс, для того чтобы попробовать ты можешь юзать л...Макс, для того чтобы попробовать ты можешь юзать любые исходники. А для продакшена покупается приватная подписка. И например некоторые маленькие софтверные компании, типа моей, хранят там исходники небольших по ее меркам продуктов. ;)Maxim Shulga (aka MaxBeard12)https://www.blogger.com/profile/05615743910272666556noreply@blogger.comtag:blogger.com,1999:blog-6346131298481535631.post-21847106547356513032013-11-13T23:00:16.991+04:002013-11-13T23:00:16.991+04:00кстати не обратил внимание: для ревью не надо выта...кстати не обратил внимание: для ревью не надо вытаскивать исходники себе локально. подтаскивается только 2 версии файла, который смотрится, для diff'аMaxim Shulga (aka MaxBeard12)https://www.blogger.com/profile/05615743910272666556noreply@blogger.comtag:blogger.com,1999:blog-6346131298481535631.post-25471595882754383682013-11-13T22:57:05.866+04:002013-11-13T22:57:05.866+04:00и будешь там хранить исходники продуктов по безопа...и будешь там хранить исходники продуктов по безопасности? ;)Maxim Shulga (aka MaxBeard12)https://www.blogger.com/profile/05615743910272666556noreply@blogger.comtag:blogger.com,1999:blog-6346131298481535631.post-90907805989074959232013-11-13T22:48:56.677+04:002013-11-13T22:48:56.677+04:00Привязка к коду комментариев даже на твоих сткринш...Привязка к коду комментариев даже на твоих сткриншотах выглядит жутко. Коментарии ,типа граждане второго сорта, где то в уголке мелким шрифтом. В Гитхабе это полноценная дискуссия как бы внутри кода. Причем как ты верно заметил ничего кроме браузер мне для этого ревью не надо.Maxim Shulga (aka MaxBeard12)https://www.blogger.com/profile/05615743910272666556noreply@blogger.comtag:blogger.com,1999:blog-6346131298481535631.post-32493394657330505532013-11-13T22:43:18.717+04:002013-11-13T22:43:18.717+04:00Кстати фраза "посмотри на новый TFS" зву...Кстати фраза "посмотри на новый TFS" звучит как издевательство, его надо ставить пол дня вникая во всякие хитрости ,тогда как сравнить тот же GitHub и BitBacket я могу за 5 минут зарегистрировав бесплатные аккаунты.Maxim Shulga (aka MaxBeard12)https://www.blogger.com/profile/05615743910272666556noreply@blogger.comtag:blogger.com,1999:blog-6346131298481535631.post-53824440443884359002013-11-05T10:02:52.563+04:002013-11-05T10:02:52.563+04:00я всегда знал, что ты кровожаден :) Но посмотри на...я всегда знал, что ты кровожаден :) Но посмотри на новый TFS, там стало лучше. До идеала далеко - но уже можно использовать.Maxim Shulga (aka MaxBeard12)https://www.blogger.com/profile/05615743910272666556noreply@blogger.comtag:blogger.com,1999:blog-6346131298481535631.post-54698595397332303412013-11-05T10:01:07.319+04:002013-11-05T10:01:07.319+04:001:1 с комментом от коллег. Да, отсутствие возможно...1:1 с комментом от коллег. Да, отсутствие возможности сделать ревью через веб при наличии этого самого веба в TFS, это серьезный прокол :(Maxim Shulga (aka MaxBeard12)https://www.blogger.com/profile/05615743910272666556noreply@blogger.comtag:blogger.com,1999:blog-6346131298481535631.post-32380466855785657142013-11-04T21:58:04.925+04:002013-11-04T21:58:04.925+04:00а еще хочу крови того творца ,который делал ветвле...а еще хочу крови того творца ,который делал ветвление в ТФС (бренчи) ... Проект с размером кода в 1Г, с 10 бренчами занимает на винте 10Г. А если бренчи заводятся порелизно, то одно изменение (багафикс) нужно делать как минимум в 2х местах . ТФС ужасен!Maxim Shulga (aka MaxBeard12)https://www.blogger.com/profile/05615743910272666556noreply@blogger.comtag:blogger.com,1999:blog-6346131298481535631.post-77399806956742506912013-11-04T21:54:57.111+04:002013-11-04T21:54:57.111+04:00Опять же, я не хочу делать это в студии (помоему т...Опять же, я не хочу делать это в студии (помоему там интеграция с ТФС сделана не для людей). Мне удобно делать это на вебе, а веб морда у ТФС (черт знает какая у нас версия) пока еще до гитхаба не дотягивает. И самое важное я не могу на вебе писать коменты к коду, а копипастить его в письма меня бесит. Вытскивать же их многогиговый проект себе чтобы сделать ревью 20 строк - мне лень.Maxim Shulga (aka MaxBeard12)https://www.blogger.com/profile/05615743910272666556noreply@blogger.comtag:blogger.com,1999:blog-6346131298481535631.post-54803736441136146422013-10-14T09:18:28.340+04:002013-10-14T09:18:28.340+04:00Я так понимаю у вас еще TFS 2010 (если не 2008). С...Я так понимаю у вас еще TFS 2010 (если не 2008). Сейчас в TFS 2012 можно настроить возможность Code Review для сторонних участников (не только команды) и посылать запросы. Со всеми вытекающими удобствами: студия, комменты привязанные к коду. Ну это для тех кто "не пацаны" и не используют GitHub ;)Maxim Shulga (aka MaxBeard12)https://www.blogger.com/profile/05615743910272666556noreply@blogger.comtag:blogger.com,1999:blog-6346131298481535631.post-64610579104598976392013-10-13T13:02:45.354+04:002013-10-13T13:02:45.354+04:00Мы уже несколько лет пользуемся для ревью кода pul...Мы уже несколько лет пользуемся для ревью кода pull реквестами в GitHub - достаточно удобно. Так же ребята из других команд присылали мне свои changeset'ы из TFS ,там вполне адекватная сравнивалка кода. Но в отличии от GitHub нельзя писать прямо в коде комментарии. Приходится копипастить кучки в письмо , что тяжко.Maxim Shulga (aka MaxBeard12)https://www.blogger.com/profile/05615743910272666556noreply@blogger.com