/irc-logs / w3c / #testing / 2014-07-16 / end

Options:

  1. # Session Start: Wed Jul 16 00:00:00 2014
  2. # Session Ident: #testing
  3. # [00:34] * Quits: plh (plehegar@public.cloak) ("Leaving")
  4. # [02:07] * Quits: darobin (rberjon@public.cloak) (Client closed connection)
  5. # [02:32] * Joins: mdas (~mdas@public.cloak)
  6. # [02:56] * Quits: lmclister (~lmclister@public.cloak) ("")
  7. # [03:13] * heycam is now known as heycam|away
  8. # [04:12] * Quits: mdas (~mdas@public.cloak) (Ping timeout: 180 seconds)
  9. # [04:28] * Joins: MK (kawamori@public.cloak)
  10. # [04:32] * Joins: mdas (~mdas@public.cloak)
  11. # [04:44] * Quits: mdas (~mdas@public.cloak) (Client closed connection)
  12. # [04:48] * Joins: mdas_ (~mdas@public.cloak)
  13. # [05:03] * Joins: lmclister (~lmclister@public.cloak)
  14. # [05:08] * Joins: darobin (rberjon@public.cloak)
  15. # [05:08] * Joins: ptressel (~chatzilla@public.cloak)
  16. # [05:13] * Joins: plh (plehegar@public.cloak)
  17. # [05:15] * Quits: darobin (rberjon@public.cloak) (Ping timeout: 180 seconds)
  18. # [05:45] * Quits: plh (plehegar@public.cloak) (Client closed connection)
  19. # [06:02] * Quits: mdas_ (~mdas@public.cloak) (Client closed connection)
  20. # [06:05] * Quits: lmclister (~lmclister@public.cloak) ("")
  21. # [06:10] * heycam|away is now known as heycam
  22. # [06:11] * Joins: mdas (~mdas@public.cloak)
  23. # [06:29] * Quits: mdas (~mdas@public.cloak) (Client closed connection)
  24. # [06:37] * Joins: lmclister (~lmclister@public.cloak)
  25. # [07:00] * Joins: glenn_ (~gadams@public.cloak)
  26. # [07:04] * Quits: glenn (~gadams@public.cloak) (Ping timeout: 180 seconds)
  27. # [07:05] * Joins: glenn (~gadams@public.cloak)
  28. # [07:08] * Quits: glenn_ (~gadams@public.cloak) (Ping timeout: 180 seconds)
  29. # [07:14] * Quits: lmclister (~lmclister@public.cloak) ("")
  30. # [07:18] * Joins: glenn_ (~gadams@public.cloak)
  31. # [07:19] * Joins: glenn__ (~gadams@public.cloak)
  32. # [07:22] * Quits: glenn (~gadams@public.cloak) (Ping timeout: 180 seconds)
  33. # [07:25] * Quits: glenn_ (~gadams@public.cloak) (Ping timeout: 180 seconds)
  34. # [07:27] * Quits: MK (kawamori@public.cloak) (Ping timeout: 180 seconds)
  35. # [07:32] * Quits: ArtB (~abarsto@public.cloak) ("Leaving.")
  36. # [07:33] * Joins: glenn (~gadams@public.cloak)
  37. # [07:38] * Quits: glenn__ (~gadams@public.cloak) (Ping timeout: 180 seconds)
  38. # [07:56] * Joins: glenn_ (~gadams@public.cloak)
  39. # [08:01] * Quits: glenn (~gadams@public.cloak) (Ping timeout: 180 seconds)
  40. # [08:04] * Quits: glenn_ (~gadams@public.cloak) (Ping timeout: 180 seconds)
  41. # [08:05] * Joins: glenn (~gadams@public.cloak)
  42. # [09:36] * heycam is now known as heycam|away
  43. # [09:45] * Joins: darobin (rberjon@public.cloak)
  44. # [09:54] * Joins: MK (kawamori@public.cloak)
  45. # [10:03] * Quits: glenn (~gadams@public.cloak) (Client closed connection)
  46. # [10:11] * Joins: Ms2ger (~Ms2ger@public.cloak)
  47. # [10:33] <jgraham> Ms2ger: Got a moment to look at https://critic.hoppipolla.co.uk/r/2056 ? It's -7 +14 so not huge.
  48. # [10:33] <jgraham> s| |/|
  49. # [10:34] * Joins: glenn (~gadams@public.cloak)
  50. # [10:35] <Ms2ger> jgraham, is there nothing in subprocess that does what you're doing in get_git_func?
  51. # [10:36] * Joins: glenn_ (~gadams@public.cloak)
  52. # [10:36] * Quits: glenn (~gadams@public.cloak) (Client closed connection)
  53. # [10:39] <jgraham> Ms2ger: check_call is close, but it returns the status code
  54. # [10:40] <jgraham> Oh wait, check_output
  55. # [10:41] <ato> There's also a few unnecessary calls to get_repo_root.
  56. # [10:41] <jgraham> ato: Hmm?
  57. # [10:41] <ato> See issue.
  58. # [10:43] <jgraham> Yeah, it does get called there. It could memorize the result or something if that's a problem
  59. # [10:43] * Quits: glenn_ (~gadams@public.cloak) (Ping timeout: 180 seconds)
  60. # [10:43] <ato> That comment isn't an issue, but it calls get_repo_root twice in main and assigns the first to an unused variable.
  61. # [10:44] <ato> Or, a variable that could be sent to create_parser, sorry.
  62. # [10:57] * Joins: AutomatedTester (~AutomatedTester@public.cloak)
  63. # [11:00] <jgraham> Yeah, that was the intention, of course.
  64. # [11:11] <ato> Why?
  65. # [11:11] <jgraham> Why what?
  66. # [11:12] <jgraham> Why was it the intention to send it to create_parser()?
  67. # [11:14] * Quits: MK (kawamori@public.cloak) (Ping timeout: 180 seconds)
  68. # [11:21] <ato> https://critic.hoppipolla.co.uk/showcomment?chain=6459
  69. # [11:21] <ato> Why first assign it to repo_root, then not pass it to create_parser?
  70. # [11:23] <Ms2ger> ato, "Yeah, [passing it to create_parser] to was the intention, of course."
  71. # [11:24] <ato> Sorry if I was being intentionally thick (-:
  72. # [11:24] * ato is multitasking folding napkins and code reviewing
  73. # [11:28] <jgraham> heh
  74. # [11:32] * Joins: MK (kawamori@public.cloak)
  75. # [11:37] * Joins: glenn (~gadams@public.cloak)
  76. # [11:44] * Quits: glenn (~gadams@public.cloak) (Ping timeout: 180 seconds)
  77. # [11:46] <AutomatedTester> i wish respec would tell you where the error was when it complains about a reference
  78. # [11:53] * Quits: MK (kawamori@public.cloak) (Ping timeout: 180 seconds)
  79. # [12:16] * Joins: abarsto (~abarsto@public.cloak)
  80. # [12:16] * abarsto is now known as ArtB
  81. # [12:24] * Joins: MK (kawamori@public.cloak)
  82. # [13:37] <darobin> AutomatedTester: if you can think of an easy way to do that that is usable by humans I'm all ears
  83. # [13:37] <darobin> by which I mean: it's reasonably easy to generate a selector/xpath to a given context, but that rarely helps that much
  84. # [13:37] <darobin> AutomatedTester: note though, if the reference doesn't exist normally just grepping for its name should be enough, no?
  85. # [13:39] * Joins: glenn (~gadams@public.cloak)
  86. # [13:39] * Joins: dom (dom@public.cloak)
  87. # [13:46] * Quits: glenn (~gadams@public.cloak) (Ping timeout: 180 seconds)
  88. # [14:00] * Joins: plh (plehegar@public.cloak)
  89. # [14:07] <MikeSmith> any opinions on https://github.com/w3c/web-platform-tests/pull/285#issuecomment-47893559 ?
  90. # [14:08] <MikeSmith> specifically, is that change worth merging after further review?
  91. # [14:20] <AutomatedTester> darobin: in this case I had the wrong casing
  92. # [14:20] <AutomatedTester> so find wasn't working
  93. # [14:21] <AutomatedTester> since it was easily overlooked
  94. # [14:21] <darobin> AutomatedTester: well, if you have suggestions to improve error reporting I'm all ears
  95. # [14:22] <darobin> the DOM isn't great at giving line numbers
  96. # [14:22] <AutomatedTester> I will add it to my todo list
  97. # [14:22] <AutomatedTester> there is code, at least in Firefox that create unique selectors to elements
  98. # [14:23] <AutomatedTester> I will see what I can borrow from that
  99. # [14:39] * Joins: glenn (~gadams@public.cloak)
  100. # [14:45] * Joins: dom_ (dom@public.cloak)
  101. # [14:47] * Quits: glenn (~gadams@public.cloak) (Ping timeout: 180 seconds)
  102. # [14:47] * Quits: dom (dom@public.cloak) (Ping timeout: 180 seconds)
  103. # [14:51] <MikeSmith> AutomatedTester wilhelm TPAC schedule has been updated to reflect the date change for the Browser Testing WG f2f http://www.w3.org/2014/11/TPAC/#Thursday/Friday%20Groups but I'm still waiting for the registration form to be updated
  104. # [14:56] <wilhelm> MikeSmith: Woho! Thanks.
  105. # [14:59] <AutomatedTester> MikeSmith: \o/
  106. # [15:21] * Joins: gitbot (~gitbot@public.cloak)
  107. # [15:21] -gitbot:#testing- [web-platform-tests] plehegar opened pull request #1110: Added template and picture elements. Added DataCue. Updated other ... (master...plh/html-idl-updates) https://github.com/w3c/web-platform-tests/pull/1110
  108. # [15:21] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  109. # [15:23] * Quits: plh (plehegar@public.cloak) ("Leaving")
  110. # [15:36] * dom_ is now known as dom
  111. # [15:40] * Joins: glenn (~gadams@public.cloak)
  112. # [15:47] * Quits: glenn (~gadams@public.cloak) (Ping timeout: 180 seconds)
  113. # [15:49] * Joins: abarsto (~abarsto@public.cloak)
  114. # [15:49] * Quits: ArtB (~abarsto@public.cloak) (Client closed connection)
  115. # [15:49] * abarsto is now known as ArtB
  116. # [15:51] * Joins: gitbot (~gitbot@public.cloak)
  117. # [15:51] -gitbot:#testing- [web-platform-tests] jgraham pushed 1 new commit to master: https://github.com/w3c/web-platform-tests/commit/95d351e59ad62441769022e198bc692b9ee5ae9d
  118. # [15:51] -gitbot:#testing- web-platform-tests/master 95d351e jgraham: Merge pull request #1109 from w3c/jgraham/manifest_no_git...
  119. # [15:51] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  120. # [15:54] * Joins: mdas (~mdas@public.cloak)
  121. # [15:55] * Quits: mdas (~mdas@public.cloak) (Client closed connection)
  122. # [15:55] * Joins: mdas (~mdas@public.cloak)
  123. # [16:37] * Joins: glenn (~gadams@public.cloak)
  124. # [16:41] <AutomatedTester> wilhelm: can I close bug 24793 as a duplicate of bug 26275 ?
  125. # [16:42] <wilhelm> AutomatedTester: Yes.
  126. # [16:43] <AutomatedTester> thanks!
  127. # [16:43] <wilhelm> 26275 is better. (c:
  128. # [16:43] <AutomatedTester> yea
  129. # [16:43] <AutomatedTester> we only have 66 open bugs... that means we can go to LC right?
  130. # [16:44] <wilhelm> Just delete all the bugs.
  131. # [16:44] <wilhelm> THen we'll be fine.
  132. # [16:44] <wilhelm> NO OPEN ISSUES LALALALA
  133. # [16:44] * Ms2ger reopens them all
  134. # [16:46] <AutomatedTester> I feel there was a w3c meme around this already
  135. # [16:47] <Ms2ger> Well, "NO OPEN ISSUES LALALALA" has a long tradition at W3C
  136. # [16:59] <wilhelm> Yes, we'd be in line with the vererable Process.
  137. # [17:01] <Ms2ger> Or "We'll fix it in the next version", and then shut down the WG
  138. # [17:04] * Quits: AutomatedTester (~AutomatedTester@public.cloak) (Client closed connection)
  139. # [17:12] * Joins: AutomatedTester (~AutomatedTester@public.cloak)
  140. # [17:13] <wilhelm> LALALALA
  141. # [17:32] * Quits: AutomatedTester (~AutomatedTester@public.cloak) (Client closed connection)
  142. # [17:39] * Joins: AutomatedTester (~AutomatedTester@public.cloak)
  143. # [17:43] * Joins: lmclister (~lmclister@public.cloak)
  144. # [18:02] * Quits: mdas (~mdas@public.cloak) (Client closed connection)
  145. # [18:02] * Joins: mdas (~mdas@public.cloak)
  146. # [18:02] * Quits: dom (dom@public.cloak) ("")
  147. # [18:09] * Quits: mdas (~mdas@public.cloak) (Ping timeout: 180 seconds)
  148. # [18:16] * Quits: darobin (rberjon@public.cloak) (Client closed connection)
  149. # [18:31] * Joins: mdas (~mdas@public.cloak)
  150. # [18:32] * Quits: MK (kawamori@public.cloak) (Ping timeout: 180 seconds)
  151. # [20:20] * Quits: AutomatedTester (~AutomatedTester@public.cloak) (Ping timeout: 180 seconds)
  152. # [20:23] * Joins: AutomatedTester (~AutomatedTester@public.cloak)
  153. # [20:49] * Quits: ptressel (~chatzilla@public.cloak) ("zzz")
  154. # [21:24] * Joins: abarsto (~abarsto@public.cloak)
  155. # [21:24] * Quits: ArtB (~abarsto@public.cloak) (Client closed connection)
  156. # [21:24] * abarsto is now known as ArtB
  157. # [21:35] * Quits: Ms2ger (~Ms2ger@public.cloak) (Ping timeout: 180 seconds)
  158. # [21:42] * Quits: mdas (~mdas@public.cloak) (Client closed connection)
  159. # [21:46] * Joins: mdas (~mdas@public.cloak)
  160. # [21:47] * Joins: Ms2ger (~Ms2ger@public.cloak)
  161. # [22:12] * Joins: gitbot (~gitbot@public.cloak)
  162. # [22:12] -gitbot:#testing- [web-platform-tests] plehegar pushed 4 new commits to master: https://github.com/w3c/web-platform-tests/compare/95d351e59ad6...fbc6d6683fe7
  163. # [22:12] -gitbot:#testing- web-platform-tests/master 81e73b0 plehegar: Added template and picture elements. Added DataCue. Updated other interfaces
  164. # [22:12] -gitbot:#testing- web-platform-tests/master 51ade06 plehegar: s/ /\t
  165. # [22:12] -gitbot:#testing- web-platform-tests/master f3bf1f4 plehegar: s/ $//
  166. # [22:12] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  167. # [22:17] * Quits: Ms2ger (~Ms2ger@public.cloak) ("nn")
  168. # [22:27] * timeless___ is now known as timeless
  169. # [22:32] * Quits: glenn (~gadams@public.cloak) (Ping timeout: 180 seconds)
  170. # [22:58] * Quits: mdas (~mdas@public.cloak) (Client closed connection)
  171. # [23:12] * Joins: plh (plehegar@public.cloak)
  172. # [23:21] * Joins: glenn (~gadams@public.cloak)
  173. # [23:36] * heycam|away is now known as heycam
  174. # [23:43] * Joins: glenn_ (~gadams@public.cloak)
  175. # [23:47] * Quits: glenn (~gadams@public.cloak) (Ping timeout: 180 seconds)
  176. # Session Close: Thu Jul 17 00:00:00 2014

The end :)