Fork Clansuite on GitHub

Internet Relay Chat - Logs for #koch
Collected by k-logbot on chat.freenode.net

IRC Log for Mittwoch, 10. Oktober 2012

  1. [00:05:51] <jakoch> https://github.com/mikey179/vfsStream/wiki/Example
  2. [00:07:02] <Rikku`on> aha
  3. [01:02:02] <Rikku`on> bin erstmal raus .. gn8
  4. [01:03:07] <jakoch> gn8
  5. [01:18:24] * Rikku`on (~Daniel@31.16.190.105) Quit (Read error: EOF from client)
  6. [06:01:09] * cslogbot (~PircBot@server.jakoch.de) has joined #clansuite
  7. [06:01:09] * Topic is 'Welcome! http://clansuite.com | http://www.youtube.com/watch?v=JPqFNQCXF9Y | https://travis-ci.org/#!/Clansuite/Clansuite'
  8. [06:01:09] * Set by Q on Sun Aug 26 02:58:29 CEST 2012
  9. [14:15:13] * Rikku`on (~Daniel@31.16.190.105) has joined #clansuite
  10. [14:19:32] <jakoch> hey
  11. [14:20:38] <Rikku`on> hey
  12. [14:24:38] <jakoch> https://travis-ci.org/#!/jakoch/doctrine2/jobs/2732759/L116
  13. [14:24:57] <jakoch> PSR2 compliance gleich 0
  14. [14:25:45] <Rikku`on> tjo, sollten die dran arbeiten :D
  15. [14:26:09] <jakoch> na ich mach das mal
  16. [14:26:33] <Rikku`on> echt?
  17. [14:26:45] <jakoch> jupp
  18. [14:27:48] <Rikku`on> haben die das denn auch irgendwie stehen das die psr2 implemtieren?
  19. [14:29:01] <Rikku`on> ich vernute mal die haben nur psr0 adoptiert, also die autoload geschichte
  20. [14:29:33] <jakoch> so wie der style aussieht isses Squiz mit Zend vermischt
  21. [14:31:46] <jakoch> für smarty hab ich das auch getestet, das kannste knicken.. das is der style den ich vor 10 monaten in clansuite hatte
  22. [14:32:00] <Rikku`on> ^^
  23. [14:32:03] <jakoch> lower_case no psr0, 1, 2
  24. [14:36:16] <Rikku`on> oha, du hast ja schon nen PR an D2 gesendet, willst du das durchziehen?
  25. [14:36:33] <jakoch> ich mach selten scherze
  26. [14:36:39] <jakoch> ^^
  27. [14:37:18] <Rikku`on> :D
  28. [14:37:24] <Rikku`on> hehe
  29. [14:37:42] <Rikku`on> damit leistet du einen großen beitrag der D2 community
  30. [14:38:38] <jakoch> ich wette das interessiert keinen und der PR wird einfach gedropped
  31. [14:39:51] <Rikku`on> frag doch beberlei ob die das wollen und ggf. mergen würden
  32. [14:45:42] <jakoch> $setCacheEntry = function($data) use ($cache, $result, $cacheKey, $realCacheKey, $queryCacheProfile) {
  33. [14:45:53] <jakoch> Expected 1 space after FUNCTION keyword; 0 found
  34. [14:46:03] <jakoch> der codesniffer is so schlecht, rofl
  35. [14:46:09] <jakoch> der letzte dreck
  36. [14:47:46] <jakoch> den muss man dann auch noch fixen
  37. [14:48:26] <jakoch> du kannst ja mal phpcs mit PSR2 gegen ZF2 laufen lassen
  38. [14:48:42] <jakoch> ich bin gespannt wie ein flitzebogen - die liste wird lang
  39. [14:56:07] <Rikku`on> die haben doch ihre eigenen coding standard, und adaptieren nur psr0 fürs autoloading
  40. [15:08:34] <Rikku`on> der phpunit coverage report in html sieht nice aus
  41. [15:08:35] <Rikku`on> http://img1.uploadscreenshot.com/images/orig/10/28223454043-orig.png
  42. [15:09:51] <jakoch> line-height beim source is mir bissi zu hoch, aber sonst bootstrap halt
  43. [15:11:51] <jakoch> normalerweise stopft man phpunit xml in php codebrowser um die lines zu sehen, das is zumindest im jenkins template so
  44. [15:17:32] <jakoch> kaffeepause, afk
  45. [15:17:37] <Rikku`on> kk
  46. [15:59:22] <jakoch> re
  47. [16:10:05] <jakoch> herrlich 3fach nested switch
  48. [16:45:27] <Rikku`on> ^^
  49. [16:52:45] <jakoch> rofl
  50. [16:53:26] <jakoch> https://github.com/doctrine/doctrine2/blob/master/lib/Doctrine/ORM/EntityRepository.php#L217
  51. [16:53:48] <jakoch> das sind die unsichtbaren fehler für die findbugs gut wäre
  52. [16:56:50] <Rikku`on> ist das nicht sogar ein syntax error?
  53. [16:57:08] <Rikku`on> bzw. der code block wird ja nie ausgeführt
  54. [16:57:20] <Rikku`on> wird wohl auch nicht von einem test gedeckt
  55. [16:58:45] <jakoch> hier würde sich nen artikel lohnen zum thema refaktorisierung von switch statements
  56. [16:59:32] <jakoch> switch scheint hier extrem beliebt zu sein
  57. [17:01:42] <jakoch> http://www.youtube.com/watch?v=IE193QLVQsA
  58. [18:55:31] <Rikku`on> sind ja nur noch < 3700 lines phpcs :D
  59. [18:55:50] <jakoch> bei uns?
  60. [18:56:13] <Rikku`on> nah, Doctrine
  61. [18:57:22] <Rikku`on> ich lösch mal den kf/phpunit branch bei mir, hab genug gespielt
  62. [18:57:35] <jakoch> stell den mal als PR
  63. [18:58:51] <Rikku`on> echt? .. ich hab da aber simple test etc rausgelöscht
  64. [18:59:18] <jakoch> na wir könnten das als parallel-branch laufen lassen
  65. [18:59:32] <jakoch> dann müsste man dort von zeit zu zeit mal syncen
  66. [19:02:28] <Rikku`on> done
  67. [19:03:55] * SyNko (~chatzilla@5.157.113.2) has joined #clansuite
  68. [19:16:38] <jakoch> ok is drin - https://github.com/KSST/KF/tree/phpunit/
  69. [19:45:43] <jakoch> afk
  70. [19:45:52] <Rikku`on> kk
  71. [20:28:07] * SyNko (~chatzilla@5.157.113.2) Quit (Quit: ChatZilla 0.9.89 [Firefox 15.0.1/20120905151427])
  72. [20:41:38] <jakoch> re
  73. [20:41:45] <Rikku`on> wb
  74. [20:44:30] <jakoch> https://github.com/doctrine/doctrine2/pull/468#r1809575
  75. [20:44:32] <jakoch> :)
  76. [20:46:55] <Rikku`on> stoffed
  77. [20:47:00] <jakoch> wie die penner klugscheissen können
  78. [20:49:53] <Rikku`on> vieleicht kannst ja bei D3 von anfang mit dabei sein, und ein auge drauf haben. denke für 2.3. und 2.4 kann man nicht alles um setzten ohne BC zu brechen
  79. [21:04:14] <Rikku`on> das stimmt etwas nicht ganz, obwohl ich das hier comitted habe : https://github.com/KSST/KF/commit/653dd76dd2dcec87dc9bc47c799d14e568e923a5 ist der --dev parameter nicht im file https://github.com/KSST/KF/blob/phpunit/.travis.yml#L15
  80. [21:05:39] <jakoch> https://github.com/KSST/KF/blob/master/.travis.yml#L15
  81. [21:05:43] <jakoch> alles da
  82. [21:06:40] <Rikku`on> der commit war gegen den phpunit branch
  83. [21:07:00] <Rikku`on> und das file welches ich linkte war auch im phpunit branch
  84. [21:08:14] <jakoch> tja.. merge fail
  85. [21:08:35] <jakoch> ich hab nur merge request geklickt...
  86. [21:09:03] <jakoch> https://github.com/KSST/KF/pull/12
  87. [21:09:08] <jakoch> das is gegen master gestellt
  88. [21:09:30] <jakoch> 13, 14 sind gegen phpunit
  89. [21:09:37] <jakoch> egal, mach einfach nochmal, wenn was fehlt
  90. [21:10:11] <Rikku`on> jup
  91. [21:10:53] <Rikku`on> done
  92. [21:13:00] <Rikku`on> die ppp queue ist total voll, dauert bissl bis ich dran bin :D
  93. [21:14:22] <jakoch> man könnte folgendes machen... die scripts die im travis direkt aufgerufen werden, in die build.xml verschieben
  94. [21:15:06] <jakoch> im travis dann nur build.xml ausführen..
  95. [21:15:21] <jakoch> dann kann ich mit der build.xml ebenfalls jenkins laufen lassen
  96. [21:18:28] <Rikku`on> keine ahnung was du meinst, aber du machst das schon :D
  97. [21:19:03] <jakoch> nen jenkins report bekommen wir sofort und ohne queue
  98. [21:19:14] <jakoch> aber der versteht leider die travis.yml nich
  99. [21:19:35] <Rikku`on> kannst auch travis forken und nen build erstellen
  100. [21:19:38] <Rikku`on> :D
  101. [21:20:11] <jakoch> ? also jenkins hat schon mehr im angebot
  102. [21:23:25] <Rikku`on> finde ich persönlich aber irgendwie schlecht. das muss man wieder maintain etc. ist ja gerade der vorteil das bei travis alles relativ einfach läuft
  103. [21:23:37] <Rikku`on> aber mach was du für richtig hälst
  104. [22:35:40] <jakoch> lol
  105. [22:35:41] <jakoch> public function load(array $criteria, $entity = null, $assoc = null, array $hints = array(), $lockMode = 0, $limit = null)
  106. [22:35:49] <jakoch> 6 params
  107. [22:35:52] <jakoch> nice eh
  108. [22:39:05] <jakoch> $value = $value[$this->em->getClassMetadata($sourceClass->associationMappings[$field]['targetEntity'])->identifier[0]];
  109. [22:41:16] <Rikku`on> solche dinger gibt es auch bei dir
  110. [22:41:38] <jakoch> nö
  111. [22:41:56] <jakoch> das glaub ich erst, wenn ich es seh
  112. [22:42:30] <Rikku`on> https://github.com/KSST/KF/blob/0c3edc5dc7cfd5511bbda61fbf5bb2fd3c960f2f/framework/Koch/Datagrid/ColumnRenderer/Link.php#L66
  113. [22:44:43] <Rikku`on> das war jetzt nur mal einer den ich gestern gesehen hatte und ein kurzen "wtf" moment hatte
  114. [22:45:40] <jakoch> hmm, ok.. das is unschön
  115. [22:46:52] <jakoch> verbesser es einfach
  116. [22:47:37] <jakoch> von wann is der link?
  117. [22:47:57] <jakoch> guter versuch https://github.com/KSST/KF/blob/master/framework/Koch/Datagrid/ColumnRenderer/Link.php
  118. [22:54:47] <jakoch> ahhh ich bin in der falschen branch
  119. [22:54:51] <jakoch> verdammt
  120. [22:54:59] <jakoch> :)
  121. [23:05:16] <jakoch> https://github.com/KSST/KF/commit/0c3edc5dc7cfd5511bbda61fbf5bb2fd3c960f2f
  122. [23:05:18] <jakoch> ah.. ok
  123. [23:05:39] <jakoch> ich hab die gleiche stelle dann auch nochmal bearbeitet
  124. [23:06:19] <jakoch> wenn function calls drin sind, dann nach möglichkeit vor die klammer ziehn
  125. [23:06:28] <jakoch> soweit das die kontrollstruktur zulässt
  126. [23:07:53] <Rikku`on> jo
  127. [23:08:10] <Rikku`on> aber sag nicht immer im vorhinein "nein"
  128. [23:08:36] <Rikku`on> wenn ich lust hätte zu suchen, würde ich sicherlich noch ein paar davon finden
  129. [23:11:04] <Rikku`on> das läuft jetzt https://travis-ci.org/#!/KSST/KF/jobs/2739395
  130. [23:11:14] <Rikku`on> abgesehen von fatal errors
  131. [23:13:52] <jakoch> gute arbeit
  132. [23:14:07] <jakoch> jo, den font_folder hab wohl nich deklariert
  133. [23:15:00] <jakoch> da fehltn s
  134. [23:15:02] <jakoch> oh mann
  135. [23:15:15] <jakoch> schön das es tests gibt
  136. [23:34:56] <jakoch> ok, also syncen von phpunit und master is way to complicated. ich ziehe simpletest rüber in ein extra repo und merge dann phpunit in master
  137. [23:42:04] <Rikku`on> ahh ok, hört sich gut an :D
  138. [23:59:02] <jakoch> hmm, mal abgesehn von den errors durch die umstellung auf phpunit
  139. [23:59:36] <jakoch> müssten eigentlich so um die 480 tests durchlaufen

Links of Mittwoch, 10. Oktober 2012

1) https://travis-ci.org/#!/KSST/KF/jobs/2739395
2) https://github.com/KSST/KF/commit/0c3edc5dc7cfd5511bbda61fbf5bb2fd3c960f2f
3) https://github.com/KSST/KF/blob/master/framework/Koch/Datagrid/ColumnRenderer/Link.php
4) https://github.com/KSST/KF/blob/0c3edc5dc7cfd5511bbda61fbf5bb2fd3c960f2f/framework/Koch/Datagrid/ColumnRenderer/Link.php#L66
5) https://github.com/KSST/KF/pull/12
6) https://github.com/KSST/KF/blob/master/.travis.yml#L15
7) https://github.com/KSST/KF/commit/653dd76dd2dcec87dc9bc47c799d14e568e923a5
8) https://github.com/KSST/KF/blob/phpunit/.travis.yml#L15
9) https://github.com/doctrine/doctrine2/pull/468#r1809575
10) https://github.com/KSST/KF/tree/phpunit/
11) http://www.youtube.com/watch?v=IE193QLVQsA
12) https://github.com/doctrine/doctrine2/blob/master/lib/Doctrine/ORM/EntityRepository.php#L217
13) http://img1.uploadscreenshot.com/images/orig/10/28223454043-orig.png
14) https://travis-ci.org/#!/jakoch/doctrine2/jobs/2732759/L116
15) http://clansuite.com
16) http://www.youtube.com/watch?v=JPqFNQCXF9Y
17) https://travis-ci.org/#!/Clansuite/Clansuite
18) https://github.com/mikey179/vfsStream/wiki/Example

These logs were automatically created by k-logbot on chat.freenode.net using the Clansuite IRC LogBot. Find the project at Github.