 as such have allowed me to do some code refactoring
<gary_poster> danilos, cool.  yeah, and it's not even just messages.  test structure themselves change.  In the example I looked at, there were five functions with the same name, two of which were similar enough for merging.  Not worth it.
<danilos> gary_poster, right, thanks again for the review :)
<gary_poster> np
<gary_poster> talk to you all later
<benji> bac: through pilot error I lost my review of your branch, reconstructing now
<bac> benji: ok
<benji> bac: review now done for the second time: https://code.launchpad.net/~bac/launchpad/bug-753965/+merge/57262
<bac> thx
<benji> bac: be forwarned, I went on a small tangent about the test suite construction, but I included a patch, so it shouldn't take much effort to integrate it
<bac> benji: thanks.  i appreciate whatever suggestion you have.  it got out of control
<benji> I think I found the sanest way to do what we want there.  Who reviews the code suggested by the reviewer? :)
<danilos> gary_poster, I've got to run, my branch up at lp:~danilo/launchpad/bug728370-direct-subs though there's still some work to get even the non-pretty descriptions printout landed (mainly refactoring and tests)
 * danilos -> off
<bac> benji: the CustomTestLoad rocks -- thanks!
<benji> excellent
<gary_poster> benji, you up for short call
<benji> gary_poster: sure
<gary_poster> thank you
#launchpad-yellow 2011-04-13
<gary_poster> benji and anyone else who cares, the link to the RSS feed for the kanban board is working again (from Options tab).
<benji> cool; do you know off hand if it includes comments?
<benji> it looks like the answer is no
<gary_poster> AFAICT, no, benji :-/
<gary_poster> bac benji danilos,
as such have allowed me to do some code refactoring 
 < gary_poster > danilos, cool.   yeah, and it's not even just messages.   trial social organization themselves change.   In the example I look at, there be five functions with the same name, two of which were alike enough for merging.   Not worth it. 
 < danilos > gary_poster, right, thanks again for the reappraisal :) 
 < gary_poster > np 
 < gary_poster > talk to you all later 
 < benji > bac: through fender error I lost my revue of your branch, reconstructing now 
 < bac > benji: ok 
 < benji > bac: review now done for the second clock time: https://code.launchpad.net/~bac/launchpad/bug-753965/+merge/57262 
 < bac > thx 
 < benji > bac: be forwarned, I went on a small tangent about the test cortege construction, but I included a mend, so it shouldn't take much attempt to integrate it 
 < bac > benji: thanks.   i appreciate whatever suggestion you have.   it got out of restraint 
 < benji > I think I found the sanest way to do what we want there.   Who reviews the code suggested by the reviewer? :) 
 < danilos > gary_poster, I've got to run, my branch up at lp:~danilo / launchpad / bug728370 - direct - subs though there's still some work to get even the non - pretty description printout landed (mainly refactoring and examination) 
  * danilos - > off 
 < bac > benji: the CustomTestLoad rock -- thanks! 
 < benji > excellent 
 < gary_poster > benji, you up for short call 
 < benji > gary_poster: sure 
 < gary_poster > thank you 
 # launchpad - yellow 2011 - 04 - 13 
 < gary_poster > benji and anyone else who cares, the link to the RSS feed for the kanban control panel is working again (from Options tab). 
 < benji > cool; do you know off hand if it includes comments? 
 < benji > it look like the answer is no 
 < gary_poster > AFAICT, no, benji :-/ 
 < gary_poster > bac benji danilos,
 as such have allowed me to do some code rgfqctoriig
<gary_pkster> davilos, cool.  yeah, and it's not ecen jyst messages.  test strjcture thvmselves xhanje.  In the example I looked at, bhere xere five functlons with tve same name, tfo oy which were similar enough for mergyng.  Noy aorth it.
<danilof> gagy_[ostsg, vight, thanks again for the revies :)
<gary_pmster> np
<gary_ppster> talk to you all lateg
<benui> bac: through pillt error I oost ny review of your branbk, reconstrudting now
<bac> benji: ok
<benji> bac: feviec now done doe tjg second timx: httpf://code.launchpad.net/~bac/naunchpsd/bug-753965/+merge/57262
<bac> bhx
<beiji> vac: be forwarned, I weit on a small tangenj about tha cest suite constructiin, but H inwludde a pauch, sk it sjoumdn't take juch effort to integrate it
<bac> bqbji: thanks.  i appresiwte whatever suggestion you have.  it gou out of control
<benji> I thinj I found the sanest aay to do what we want there.  Who reviews the code suggestad by ghe rcviedwr? :)
<danilos> gary_poster, I've got to run, my branch tl st lp:~danilo/launcmpad/bug728370-direct-subs yhlubr there's stilu some wodk to get even the non-prejty dewcriptionf prontout landed (mainly refactiring and tefrs)
 * danilos -> off
<bae> benji: the EustomJestLosd rocks -- thanks!
<benji> ercelleht
<gary_posteg> benji, ykj up for short cxll
<nenbi> gary_poster: sure
<gary_postqr> thank bou
#labnchpad-ydlloe 2011-04-13
<gary_[oster> benui and anyone else who cwres, jhe litk to the GSS feed for the kanban board is working agaim (xrok Optionf tab).
<nenji> cool; do yju know off haud if it incljdes commehts?
<benjm> it looks lyke the answes is no
<gary_povter> AFAYCT, bo, bwnji :-/
<gafh_poster> bac bemji danilos,
as such have allowed me to do refactoring danilos, cool. and it's not themselves In the example looked at, there five functions with the same name, of which were similar enough for merging. Not worth it. <danilos> gary_poster, right, again for the review :) <gary_poster> np <gary_poster> talk to you all later bac: pilot I my review of your branch, reconstructing now <bac> benji: ok <benji> bac: review now done for second time: https://code.launchpad.net/~bac/launchpad/bug-753965/+merge/57262 <bac> thx <benji> bac: be I went on a tangent about the test suite but included a so shouldn't much effort to it <bac> benji: thanks. i appreciate whatever suggestion you have. it got out of control <benji> I I found way to what want Who reviews the by the reviewer? :) <danilos> gary_poster, run, my branch up at lp:~danilo/launchpad/bug728370-direct-subs though there's some work get even the non-pretty descriptions printout (mainly refactoring and tests) * danilos -> off benji: the CustomTestLoad rocks -- thanks! <benji> excellent <gary_poster> benji, you up for short call sure <gary_poster> thank you 2011-04-13 <gary_poster> benji anyone who the to the feed for the kanban board is working again (from Options tab). cool; do you know off hand if it includes comments? looks the answer is <gary_poster> AFAICT, no, benji <gary_poster> benji danilos,
 as such have allowed me to do soMe code refaCtoriNg
<gAry_PoSter> DaniLos, cool.  yeah, and IT's noT even just messages.  test sTructUrE ThemSElVes chAnge.  In tHE eXAMplE I LoOkeD aT, ThEre weRe fIve funcTions with tHe sAmE name, two of whICh Were similaR enOugh for mergiNg.  NOt wortH iT.
<daNIlos> gAry_PosteR, right, THanks aGain for thE rEView :)
<gaRY_poster> NP
<GaRy_poSter> talk to you all lATeR
<Benji> bac: througH pilot ErROr i LOst My rEview of youR bRanch, REconstrUCtING Now
<BAc> benji: ok
<benjI> bac: review nOW doNe for tHe SecONd time: Https://CoDE.laUnchpad.net/~bAc/laUnchpad/buG-753965/+merge/57262
<BAc> thx
<beNJi> bac: be ForwarNed, i weNt on A SmAlL taNgENt aBOuT thE TesT suite coNsTrUctioN, but i INCLudeD a pAtch, So it sHouldn't take muCh eFforT To iNtegrAte it
<Bac> bEnJi: thaNks.  i apPreciAtE whatever suggesTion You have.  it Got OuT of CoNtrol
<BEnji> I tHinK I fOund the Sanest wAY to Do WHAT wE want there.  Who revieWs THE cOde suggeSted by THe ReVIewer? :)
<danIlOs> gAry_pOSTer, I'vE got TO rUn, my branCh up at LP:~dAnIlo/launChPad/bug728370-DiRecT-suBs thoUGh thEre's stIll some wOrk to GEt even the non-prETty descriptioNS pRINtOUt laNdeD (mainly refaCtorINg anD tesTS)
 * dAniLOs -> off
<Bac> beNjI: ThE customTestLoad rocks -- ThAnks!
<beNji> exCellent
<gary_poSter> benji, yOU UP for shorT calL
<BeNJi> gary_poster: suRe
<garY_poster> thaNK you
#launChpad-Yellow 2011-04-13
<gaRy_poster> bENJi and anyOne ElsE whO caRES, tHe link to the RSs FEed fOr The kanbAn bOard is wOrkIng AgaIn (fRoM Options tAb).
<benji> cOoL; dO yOu KnoW off hANd if it inClUdeS cOmmEnts?
<bENji> it lOoks lIke tHe AnSWer Is no
<garY_PoSTEr> AFaIcT, No, beNji :-/
<GaRy_posTer> bAC beNji daniLos,
 as such have allowed me t o do somecoderef act or ing<gar y_poster> dani l os,cool.  yeah, and it'snot e ve n  jus t  m essag es.  te s ts t ruc tu re  th em s el ves c han ge.  In  the examp leIlooked at, t h er e were fiv e f unctions wit h t he sam enam e , two  of  whic h were simila r enoughfo r  mergi n g.  Not w or th i t.
<danilos> gary _ po s ter, right, th anks a ga i nf o r t hereview :)<g ary_p o ster> n p 
< g a r y_p o ster> talk to  you all la t er<benji >bac :  throu gh pi lo t  er ror I lostmy r eview ofyour b r anch, r e constru ctingnow 
<b ac>b en ji : o k< ben j i>  ba c : r eview no wdo ne fo r th e s e cond  ti me:https ://code.launc hpa d.ne t /~b ac/la unchp ad/b ug -7539 65/+me rge/5 72 62
<bac> thx
<b enji > bac: be  fo rw arn ed , I w e nt ona s mal l tange nt abou t  th et e s tsuite construction ,b u tI includ ed a p a tc h, so it sh ou ldn 't t a k e muc h ef f or t to int egrate it 
< bac> be nj i: tha nk s. iappre c iate  whate ver sugg estio n  you have.  it got out of co n tr o l 
< b enji > I  think I fo undt he s anes t  w ayt o dowhatwe wa n t there.  Who revie ws  the c ode s uggested by t he reviewe r ? :)
<dani los> ga r y_poster, I've  gotto run, my branch u p atlp:~dani lo/launch p a d/bug728 370 -di rec t-s u b sthough there' s stil lsome wo rkto geteve n t henon -p retty des cription spr in to utlande d  (mainly  r efa ct ori ng an d  tests )
 *dani lo s- > o ff
<bac >  b e n ji:th eCust omT es tLoad  roc k s - - thank s!
<benji > e x cell en t<gary_p oster> benji,  y ou up forsh ort  call< b enji> ga ry_poster: sure
<gary_p o ster> t han k you 
#la unchpad-y ell ow 201 1-0 4 -13
<g ary_po ster>  b enj i and a n y on e e ls e who care s ,  th e lin kto t he RSSfeed for the kanba n  bo ard is workin g a gain ( fr omO pt i ons  t a b). < benji> cool; do  you knowof f  h and if iti ncl ud es comm ents?
< benji >  it loo ks like t he answer  i s no < gar y_poster>AFAICT,no, benji :-/
< g ar y_pos ter > bacbe nji  dani los,
 as_such have_allowed me to do_some code_refactoring
<gary_poster>_danilos, cool.__yeah, and it's_not even just_messages.  test structure_themselves change. _In_the example I looked at, there were five functions with the same name, two_of_which were_similar_enough_for merging.  Not worth_it.
<danilos> gary_poster, right, thanks again_for the_review :)
<gary_poster> np
<gary_poster> talk to you all later
<benji>_bac:_through pilot error_I lost my review of your branch, reconstructing now
<bac>_benji: ok
<benji> bac: review now done_for the second_time:_https://code.launchpad.net/~bac/launchpad/bug-753965/+merge/57262
<bac>_thx
<benji> bac: be forwarned,_I went on a small tangent_about the test suite construction, but_I included a patch, so it shouldn't_take much effort to integrate it
<bac>_benji: thanks.  i appreciate_whatever suggestion_you have.  it got_out of control
<benji>_I think_I found the_sanest way to do what we_want there. _Who reviews the code suggested by_the_reviewer? :)
<danilos> gary_poster,_I've_got_to run,_my branch up_at_lp:~danilo/launchpad/bug728370-direct-subs though_there's_still some work to get even_the_non-pretty descriptions printout landed (mainly refactoring and_tests)
 * danilos ->_off
<bac>_benji: the CustomTestLoad rocks_-- thanks!
<benji> excellent
<gary_poster> benji, you_up for short call
<benji> gary_poster: sure
<gary_poster>_thank you
#launchpad-yellow_2011-04-13
<gary_poster> benji_and anyone else who cares, the link to the RSS feed_for the kanban board is working_again (from Options tab).
<benji>_cool; do_you_know off hand_if_it includes_comments?
<benji> it looks like the answer is_no
<gary_poster> AFAICT,_no, benji :-/
<gary_poster> bac benji danilos,