Friday, 2013-08-23

*** tpb has joined #timvideos00:00
*** shenki_ is now known as shenki01:15
*** mithro-laptop has quit IRC01:44
*** iiie0 has quit IRC02:28
Sewariiie: meeting?02:56
*** mithro-laptop has joined #timvideos02:59
iiiehm, my button to get into chat in gmail seems to be hiding02:59
iiiecould help if I were in the right account...03:00
*** mithro-laptop has quit IRC03:15
SewarThanks for the meeting iiie03:30
iiieAnd thank you!03:30
Sewar'transaction.commit_on_success()' fixed Selenium issues :D04:03
Sewarthanks iiie04:05
*** hyades has joined #timvideos05:13
mithroHey Sewar, have you been running any type of linter over your code?06:37
Sewarmithro: rarely, because it has a lot of false positives06:40
mithroSewar, oh?06:40
mithroSewar, if your code has lots of false positives - generally there is something wrong with your style?06:40
mithrowhich linter are you using?06:41
SewarI need to find Django tailored lintrc06:41
SewarE1101:605,8:DetailedTimingForm.__init__: Instance of 'DetailedTimingForm' has no 'fields' member06:42
tpbTitle: django-lint: Static analysis tool for Django projects lamby (at
mithroSewar, i'm sure #django might be able to point you in the right direction06:45
Sewarthat looks nice, should give it a try06:45
mithroSewar, also - what about pep8?06:45
Sewarmithro, I have pep8 in my editor and try to comply to it06:46
tpb<> (at
mithroSewar, it would be good to add it as part of the travis-ci build06:47
Sewarmithro, and where can I pipe the output to? or just have it there in the build log?06:48
mithrojust have it in the build log06:48
Sewarsounds good06:48
mithroyour code should be pep8 clean06:48
Sewarit's, just checked06:49
mithroyou can use # noqa is you need a violation for some reason06:49
mithroSewar, great! so make the build fail if pep8 reports any issues :)06:49
Sewarok :)06:50
mithroyou probably want to not use             self.browser.find_element_by_xpath06:51
mithroSewar, either use the id or css class06:52
mithrothat way when the page structure changes you won't have test failures06:52
Sewarwell, if the code changes, tests need to be updated too?06:54
mithrocss classes are useful because they are also what is used to display things06:54
mithroSewar, tests shouldn't need updating 99% of the time06:54
mithrounless you actually change functionality06:54
mithroIE your test is trying to make sure you don't break functionality right06:57
mithrosee ya!06:57
*** mithro-laptop has joined #timvideos12:08
*** hyades has quit IRC12:41
*** hyades has joined #timvideos13:25
*** hyades has quit IRC15:14
*** hyades has joined #timvideos15:17
*** hyades has quit IRC15:47
*** hyades has joined #timvideos16:13
*** hyades has quit IRC16:20
*** hyades has joined #timvideos16:22
*** hyades has quit IRC16:49
*** hyades has joined #timvideos16:49
*** hyades has quit IRC17:03
*** hyades has joined #timvideos17:11
*** hyades has quit IRC17:17
*** hyades has joined #timvideos17:38
*** mithro-laptop has quit IRC17:42
*** hyades has quit IRC17:49
*** hyades has joined #timvideos18:03
*** hyades has quit IRC18:34
*** hyades has joined #timvideos18:47
*** hyades has quit IRC19:09

Generated by 2.12.1 by Marius Gedminas - find it at!