Comments on: Code-review, czyli przegląd kodu http://blog.mocna-kawa.com/2007/10/code-review-czyli-przeglad-kodu/ radosny blog o nowych technologiach Tue, 03 Jan 2012 21:57:12 +0000 hourly 1 http://wordpress.org/?v=3.0.1 By: Pawel Szulc http://blog.mocna-kawa.com/2007/10/code-review-czyli-przeglad-kodu/comment-page-1/#comment-263 Pawel Szulc Thu, 28 May 2009 07:48:07 +0000 http://blog.mocna-kawa.com/2007/10/22/code-review-czyli-przeglad-kodu/#comment-263 no i co klepnales to w koncu? :) no i co klepnales to w koncu? :)

]]> By: Dominik http://blog.mocna-kawa.com/2007/10/code-review-czyli-przeglad-kodu/comment-page-1/#comment-240 Dominik Thu, 19 Mar 2009 15:05:47 +0000 http://blog.mocna-kawa.com/2007/10/22/code-review-czyli-przeglad-kodu/#comment-240 Przejrzyj sobie narzędzia atlassiana, zwłaszcza: http://www.atlassian.com/software/crucible/ Przejrzyj sobie narzędzia atlassiana, zwłaszcza:
http://www.atlassian.com/software/crucible/

]]> By: Tomek http://blog.mocna-kawa.com/2007/10/code-review-czyli-przeglad-kodu/comment-page-1/#comment-72 Tomek Mon, 29 Oct 2007 18:07:04 +0000 http://blog.mocna-kawa.com/2007/10/22/code-review-czyli-przeglad-kodu/#comment-72 To konca kwietnia chcesz napisac ? Dobre ! No ale probuj... ;-) To konca kwietnia chcesz napisac ? Dobre !

No ale probuj… ;-)

]]> By: J. http://blog.mocna-kawa.com/2007/10/code-review-czyli-przeglad-kodu/comment-page-1/#comment-71 J. Sat, 27 Oct 2007 21:13:43 +0000 http://blog.mocna-kawa.com/2007/10/22/code-review-czyli-przeglad-kodu/#comment-71 Pomysł oki, prezentacja nie do przyjecia. Pomysł oki, prezentacja nie do przyjecia.

]]> By: Marcin http://blog.mocna-kawa.com/2007/10/code-review-czyli-przeglad-kodu/comment-page-1/#comment-70 Marcin Thu, 25 Oct 2007 22:26:51 +0000 http://blog.mocna-kawa.com/2007/10/22/code-review-czyli-przeglad-kodu/#comment-70 Szacunek co do prezentacji:) Fajnie, ze postanowiles z tym zawalczyc. Tak sie sklada, ze w obecnym projekcie robimy przeglady "nieblokujace" typu "na wyrywki", bo najzwyczajniej nie ma czasu na doglebna analize kazdego checkina. I tak sie ciesze, ze udalo mi sie przekonac Klienta co do zalet tego typu rozwiazania (chociaz troche okrojonego). Jesli ktos zakomituje kod z razacymi bledami (nie chodzi mi w tym przypadku raczej o problemy typu nazewnictwo) to po review z wynikiem negatywnym poprawia je i umieszcza w repozytorium ponownie. W sumie to dziala. Mam problem z jednym czlonkiem zespolu, ktory kazda krytyke przyjmuje bardzo osobiscie i niepotrzebnie sie stresuje, marnujac czas na udawadnienie, ze to nie jego wina itp. Mysle, ze krotki rozdzial o sposobie postepowania w takich przypadkach bylby calkiem ciekawy. Z niecierpliwoscia czekam na dalsze postepy (a przede wszytkim wynikowy soft). Na jutrzejszej konferencji mnie nie bedzie (deadline :/ ), ale moze wieczorem... Szacunek co do prezentacji:) Fajnie, ze postanowiles z tym zawalczyc. Tak sie sklada, ze w obecnym projekcie robimy przeglady “nieblokujace” typu “na wyrywki”, bo najzwyczajniej nie ma czasu na doglebna analize kazdego checkina. I tak sie ciesze, ze udalo mi sie przekonac Klienta co do zalet tego typu rozwiazania (chociaz troche okrojonego). Jesli ktos zakomituje kod z razacymi bledami (nie chodzi mi w tym przypadku raczej o problemy typu nazewnictwo) to po review z wynikiem negatywnym poprawia je i umieszcza w repozytorium ponownie. W sumie to dziala. Mam problem z jednym czlonkiem zespolu, ktory kazda krytyke przyjmuje bardzo osobiscie i niepotrzebnie sie stresuje, marnujac czas na udawadnienie, ze to nie jego wina itp. Mysle, ze krotki rozdzial o sposobie postepowania w takich przypadkach bylby calkiem ciekawy. Z niecierpliwoscia czekam na dalsze postepy (a przede wszytkim wynikowy soft). Na jutrzejszej konferencji mnie nie bedzie (deadline :/ ), ale moze wieczorem…

]]> By: Wiktor http://blog.mocna-kawa.com/2007/10/code-review-czyli-przeglad-kodu/comment-page-1/#comment-69 Wiktor Wed, 24 Oct 2007 10:46:44 +0000 http://blog.mocna-kawa.com/2007/10/22/code-review-czyli-przeglad-kodu/#comment-69 Jak najbardziej mój projekt będzie upubliczniony. Mam nadzieje, że będę mógł go pokazać już na początku lutego (oby oby). Dzięki za znalezienie literówki. Jest to mój błąd i jest on już poprawiony. Co do czcionek to używam mniejszych czcionek w podpunktach (ich kolor jest szarawy). Jest to moje świadome działanie, może faktycznie na przyszłość powinienem to zmienić. Dzięki! (mam nadzieję, że slideshare zaraz uaktualni prezentację ;)) Jak najbardziej mój projekt będzie upubliczniony. Mam nadzieje, że będę mógł go pokazać już na początku lutego (oby oby).

Dzięki za znalezienie literówki. Jest to mój błąd i jest on już poprawiony.

Co do czcionek to używam mniejszych czcionek w podpunktach (ich kolor jest szarawy). Jest to moje świadome działanie, może faktycznie na przyszłość powinienem to zmienić.

Dzięki!

(mam nadzieję, że slideshare zaraz uaktualni prezentację ;) )

]]> By: Piotrek http://blog.mocna-kawa.com/2007/10/code-review-czyli-przeglad-kodu/comment-page-1/#comment-68 Piotrek Wed, 24 Oct 2007 06:30:06 +0000 http://blog.mocna-kawa.com/2007/10/22/code-review-czyli-przeglad-kodu/#comment-68 Po pierwsze: na 18 slajdzie jest "Dwie pary oczy..." rozumiem, że to literówka - mi wydaje się, że powinno być "oczu". Po drugie: rozmiary czcionek nie są jednolite. Mam wrażenie - może błędne - że część zdań na tym samym poziomie wypunktowania na jedyn slajdzie są pisane czcionką w rozmiarze X, a na innych X--. Jeśli nie możesz zmieścić treści na jednym slajdzie, to go podziel na kilka. Po trzecie: podoba mi się temat i treść pracy :) Naprawdę jestem ciekaw co z tego wyjdzie. Pytanie: Czy upublicznisz ten projekt? Pozdrawiam Piotrek Po pierwsze: na 18 slajdzie jest “Dwie pary oczy…” rozumiem, że to literówka – mi wydaje się, że powinno być “oczu”.
Po drugie: rozmiary czcionek nie są jednolite. Mam wrażenie – może błędne – że część zdań na tym samym poziomie wypunktowania na jedyn slajdzie są pisane czcionką w rozmiarze X, a na innych X–. Jeśli nie możesz zmieścić treści na jednym slajdzie, to go podziel na kilka.
Po trzecie: podoba mi się temat i treść pracy :) Naprawdę jestem ciekaw co z tego wyjdzie.

Pytanie: Czy upublicznisz ten projekt?
Pozdrawiam
Piotrek

]]> By: wiktor http://blog.mocna-kawa.com/2007/10/code-review-czyli-przeglad-kodu/comment-page-1/#comment-67 wiktor Tue, 23 Oct 2007 13:37:32 +0000 http://blog.mocna-kawa.com/2007/10/22/code-review-czyli-przeglad-kodu/#comment-67 Tak, przedstawiłem taką prezentację na seminarium. Pamiętaj, że wszystko zależy od tego, kto jest prowadzącym i jakie ma podejście do tego. Celem mojej mgr jest stworzenie aplikacji, a reszta to dla mnie jest na prawdę papierkowa robota, bo nie stanowi meritum mgr. Zgadzam się, że występuje w niej "luzacki ton" i jest on przeze mnie wprowadzony świadomie, aby publiczność nie zasypiała :). Natomiast kiedy opowiadałem o moim pomyśle to na taki luz już sobie nie pozwalałem. Tak, przedstawiłem taką prezentację na seminarium. Pamiętaj, że wszystko zależy od tego, kto jest prowadzącym i jakie ma podejście do tego. Celem mojej mgr jest stworzenie aplikacji, a reszta to dla mnie jest na prawdę papierkowa robota, bo nie stanowi meritum mgr. Zgadzam się, że występuje w niej “luzacki ton” i jest on przeze mnie wprowadzony świadomie, aby publiczność nie zasypiała :) . Natomiast kiedy opowiadałem o moim pomyśle to na taki luz już sobie nie pozwalałem.

]]> By: Rafał http://blog.mocna-kawa.com/2007/10/code-review-czyli-przeglad-kodu/comment-page-1/#comment-66 Rafał Tue, 23 Oct 2007 13:26:40 +0000 http://blog.mocna-kawa.com/2007/10/22/code-review-czyli-przeglad-kodu/#comment-66 przedstawiles taką prezentację na seminarium? wydaje mi sie, ze jej luzacki ton ("bajery", "papierkowa robota") nie bardzo pasuje na uczlenię i pokazywanie jej profesorowi. przedstawiles taką prezentację na seminarium? wydaje mi sie, ze jej luzacki ton (“bajery”, “papierkowa robota”) nie bardzo pasuje na uczlenię i pokazywanie jej profesorowi.

]]> By: wiktor http://blog.mocna-kawa.com/2007/10/code-review-czyli-przeglad-kodu/comment-page-1/#comment-65 wiktor Tue, 23 Oct 2007 09:56:33 +0000 http://blog.mocna-kawa.com/2007/10/22/code-review-czyli-przeglad-kodu/#comment-65 Ooo to coś nowego, nie widziałem tego jeszcze. Dzięki za linka :) Ooo to coś nowego, nie widziałem tego jeszcze. Dzięki za linka :)

]]>