/irc-logs / w3c / #testing / 2014-04-09 / end

Options:

  1. # Session Start: Wed Apr 09 00:00:00 2014
  2. # Session Ident: #testing
  3. # [00:06] * Quits: lukeis (~Adium@public.cloak) ("Leaving.")
  4. # [00:08] * Joins: rhauck (~Adium@public.cloak)
  5. # [00:44] * Joins: gitbot (~gitbot@public.cloak)
  6. # [00:44] -gitbot:#testing- [web-platform-tests] sideshowbarker pushed 1 new commit to submission/sideshowbarker/datetime: https://github.com/w3c/web-platform-tests/commit/a4b8200e6c48137b4606d567e69acfb0eea664d3
  7. # [00:44] -gitbot:#testing- web-platform-tests/submission/sideshowbarker/datetime a4b8200 Michael[tm] Smith: Added a case of year < 1000 + a case of year > 3000
  8. # [00:44] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  9. # [00:46] * Joins: zcorpan (~zcorpan@public.cloak)
  10. # [00:54] * Quits: zcorpan (~zcorpan@public.cloak) (Ping timeout: 180 seconds)
  11. # [00:57] * Quits: jgraham (~jgraham@public.cloak) (Client closed connection)
  12. # [00:59] * Joins: jgraham (~jgraham@public.cloak)
  13. # [01:17] * Quits: lmcliste_ (~lmclister@public.cloak) ("")
  14. # [01:53] * Quits: JohnJansen (~JohnJansen@public.cloak) (Ping timeout: 180 seconds)
  15. # [02:01] * Quits: darobin (rberjon@public.cloak) (Client closed connection)
  16. # [02:01] * Quits: plh (plehegar@public.cloak) ("Leaving")
  17. # [02:03] * Quits: ptressel (~chatzilla@public.cloak) ("zzz")
  18. # [02:26] * Quits: rhauck (~Adium@public.cloak) ("Leaving.")
  19. # [02:27] * Quits: anssik (~uid10742@public.cloak) ("Connection closed for inactivity")
  20. # [02:33] * Quits: AutomatedTester (~AutomatedTester@public.cloak) (Client closed connection)
  21. # [02:48] * Joins: zcorpan (~zcorpan@public.cloak)
  22. # [02:49] * Quits: zcorpan (~zcorpan@public.cloak) (Client closed connection)
  23. # [02:49] * Joins: zcorpan (~zcorpan@public.cloak)
  24. # [02:57] * Quits: zcorpan (~zcorpan@public.cloak) (Ping timeout: 180 seconds)
  25. # [02:59] * Joins: yankhates_cb (~yankhatescb@public.cloak)
  26. # [03:28] * Quits: ArtB (~abarsto@public.cloak) ("Leaving.")
  27. # [03:50] * Joins: zcorpan (~zcorpan@public.cloak)
  28. # [03:57] * Quits: zcorpan (~zcorpan@public.cloak) (Ping timeout: 180 seconds)
  29. # [04:51] * Joins: zcorpan (~zcorpan@public.cloak)
  30. # [04:58] * Quits: zcorpan (~zcorpan@public.cloak) (Ping timeout: 180 seconds)
  31. # [05:05] * Quits: yankhates_cb (~yankhatescb@public.cloak) (Ping timeout: 180 seconds)
  32. # [05:52] * Joins: zcorpan (~zcorpan@public.cloak)
  33. # [05:59] * Quits: zcorpan (~zcorpan@public.cloak) (Ping timeout: 180 seconds)
  34. # [06:05] * Quits: scott_gonzalez (~scott_gonzalez@public.cloak) (Client closed connection)
  35. # [06:05] * Joins: scott_gonzalez (~scott_gonzalez@public.cloak)
  36. # [06:29] * Quits: mdyck (~mdyck@public.cloak) ("Leaving")
  37. # [06:30] * Quits: denis (denis@public.cloak) ("ZNC - http://znc.in")
  38. # [06:34] * Joins: denis (denis@public.cloak)
  39. # [06:37] * Joins: zcorpan (~zcorpan@public.cloak)
  40. # [06:40] * Joins: zcorpan_ (~zcorpan@public.cloak)
  41. # [06:44] * Quits: zcorpan (~zcorpan@public.cloak) (Ping timeout: 180 seconds)
  42. # [07:04] * Joins: yankhates_cb (~yankhatescb@public.cloak)
  43. # [08:00] * Joins: darobin (rberjon@public.cloak)
  44. # [08:43] * Quits: zcorpan_ (~zcorpan@public.cloak) (Client closed connection)
  45. # [08:49] * Joins: zcorpan (~zcorpan@public.cloak)
  46. # [08:57] * Quits: scott_gonzalez (~scott_gonzalez@public.cloak) (Client closed connection)
  47. # [08:57] * Joins: scott_gonzalez (~scott_gonzalez@public.cloak)
  48. # [09:00] * Quits: Lachy (~Lachy@public.cloak) ("My MacBook Pro has gone to sleep. ZZZzzz…")
  49. # [09:09] * Joins: Jazcash (~Jazcash@public.cloak)
  50. # [09:10] * Parts: Jazcash (~Jazcash@public.cloak) (Jazcash)
  51. # [09:10] * Quits: yankhates_cb (~yankhatescb@public.cloak) (Ping timeout: 180 seconds)
  52. # [09:12] * Joins: yankhates_cb (~yankhatescb@public.cloak)
  53. # [09:28] * Joins: gitbot (~gitbot@public.cloak)
  54. # [09:28] -gitbot:#testing- [web-platform-tests] yunxiaoxie opened pull request #841: Put authorship information into IndexedDB test files (master...yunxiaoxie/IndexedDB-authorship) https://github.com/w3c/web-platform-tests/pull/841
  55. # [09:28] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  56. # [10:20] * Disconnected
  57. # [10:22] * Attempting to rejoin channel #testing
  58. # [10:22] * Rejoined channel #testing
  59. # [10:22] * Topic is '"also, I hope no one actually reads this, but I have to admit that I've kind of gotten used to Critic" --darobin'
  60. # [10:22] * Set by plh on Mon Mar 31 21:50:34
  61. # [10:26] * Joins: shepazu (schepers@public.cloak)
  62. # [12:09] * jgraham has regenerated the ssl cert for critic
  63. # [12:13] * Joins: AutomatedTester (~AutomatedTester@public.cloak)
  64. # [12:16] <jgraham> zcorpan: So the problem with the new query-encoding tests seems to be that they hang Chrome. At least on my machine
  65. # [12:21] <zcorpan> jgraham: don't hang for me
  66. # [12:22] <jgraham> zcorpan: Then I blame my Chrome unstable build for being unstable
  67. # [12:23] <zcorpan> 36.0.1932.0 canary over here
  68. # [12:23] <jgraham> This is 35. something
  69. # [12:23] <jgraham> I'm updating
  70. # [12:23] <jgraham> Anyway if it works for you, that's good enough for me
  71. # [12:24] <zcorpan> yep should be ok
  72. # [12:57] * Quits: SimonSapin (~simon@public.cloak) ("WeeChat 0.3.8")
  73. # [13:00] * Joins: SimonSapin (~simon@public.cloak)
  74. # [13:00] * Joins: ptressel (~chatzilla@public.cloak)
  75. # [13:01] * Parts: SimonSapin (~simon@public.cloak) (WeeChat 0.3.8)
  76. # [13:35] * Joins: Lachy (~Lachy@public.cloak)
  77. # [13:36] * Joins: gitbot (~gitbot@public.cloak)
  78. # [13:36] -gitbot:#testing- [web-platform-tests] jgraham pushed 1 new commit to jgraham/query_encoding_stash: https://github.com/w3c/web-platform-tests/commit/0a32beacdef0c19d6d49cd1be732f9165fb5a17e
  79. # [13:36] -gitbot:#testing- web-platform-tests/jgraham/query_encoding_stash 0a32bea James Graham: fixup! Poll for resource loading on the client side rather than the server side.
  80. # [13:36] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  81. # [13:36] <jgraham> zcorpan: OK, I *think* it
  82. # [13:36] <jgraham> 's stable now
  83. # [13:43] * Joins: gitbot (~gitbot@public.cloak)
  84. # [13:43] -gitbot:#testing- [web-platform-tests] ayg closed pull request #838: Remove incorrect setAttribute() test (master...submission/ayg-remove-setattribute-test) https://github.com/w3c/web-platform-tests/pull/838
  85. # [13:43] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  86. # [13:46] * Joins: dom (dom@public.cloak)
  87. # [13:51] * Joins: gitbot (~gitbot@public.cloak)
  88. # [13:51] -gitbot:#testing- [web-platform-tests] ayg opened pull request #842: Fix incorrect setAttribute test (master...submission/ayg-fix-setattribute-test) https://github.com/w3c/web-platform-tests/pull/842
  89. # [13:51] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  90. # [13:58] * Joins: gitbot (~gitbot@public.cloak)
  91. # [13:58] -gitbot:#testing- [web-platform-tests] jgraham pushed 1 new commit to jgraham/query_encoding_stash: https://github.com/w3c/web-platform-tests/commit/9e1f53182b80d8c890257a03fe9086b5530e7f04
  92. # [13:58] -gitbot:#testing- web-platform-tests/jgraham/query_encoding_stash 9e1f531 James Graham: fixup! Poll for resource loading on the client side rather than the server side.
  93. # [13:58] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  94. # [14:00] * Quits: Lachy (~Lachy@public.cloak) ("Textual IRC Client: www.textualapp.com")
  95. # [14:18] * Joins: plh (plehegar@public.cloak)
  96. # [14:46] * Joins: gitbot (~gitbot@public.cloak)
  97. # [14:46] -gitbot:#testing- [web-platform-tests] jgraham pushed 1 new commit to master: https://github.com/w3c/web-platform-tests/commit/86ce2448953dc8b43c5ade3f0e604bc8832d0970
  98. # [14:46] -gitbot:#testing- web-platform-tests/master 86ce244 jgraham: Merge pull request #839 from w3c/jgraham/query_encoding_stash...
  99. # [14:46] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  100. # [14:52] * Quits: zcorpan (~zcorpan@public.cloak) (Client closed connection)
  101. # [14:53] * Joins: zcorpan (~zcorpan@public.cloak)
  102. # [15:00] * Quits: zcorpan (~zcorpan@public.cloak) (Ping timeout: 180 seconds)
  103. # [15:18] <MikeSmith> jgraham: so at some point I'd like to know what the proper steps are I should follow when I want to squash a set of changes and rebase before pushing to master
  104. # [15:18] * Joins: Ms2ger (~Ms2ger@public.cloak)
  105. # [15:19] <MikeSmith> the way I've been doing it is, after a review is accepted I do the squash but don't push it to the review branch again
  106. # [15:19] <MikeSmith> I just push it directly to master from the review branch
  107. # [15:20] <MikeSmith> git push origin foo-branch:master
  108. # [15:20] <MikeSmith> and then I manually close the PR
  109. # [15:21] <MikeSmith> but of course what happens then is that in github it looks like the PR branch was never merged
  110. # [15:21] <Ms2ger> Indeed
  111. # [15:21] <MikeSmith> and then critic thinks it was "Dropped"
  112. # [15:21] <MikeSmith> which is all fine by me personally
  113. # [15:22] <MikeSmith> but I guess it's not consistent with the workflow for the rest of the PRs
  114. # [15:23] <Ms2ger> I long for a Merge+autosquash button on gihub
  115. # [15:23] <Ms2ger> github
  116. # [15:23] <MikeSmith> yeah
  117. # [15:23] <MikeSmith> that's the way it should work
  118. # [15:25] <MikeSmith> but anyway if I could just push the squashed change to the review branch that would get github to deal with it normally
  119. # [15:27] <MikeSmith> I thought there wasn't any way to do that without wedging critic but I recall jgraham saying here or somewhere a few days ago that there is actually some right way to do that so that critic doesn't end up up wedged
  120. # [15:27] <jgraham> You can do that. Critic might get upset ut it doesn't really matter
  121. # [15:27] <jgraham> Or you can tell critic about the rebase
  122. # [15:27] <MikeSmith> ok
  123. # [15:28] <MikeSmith> how do I tell critic about the rebase?
  124. # [15:28] <Ms2ger> Even about a squash?
  125. # [15:48] * Joins: MK (kawamori@public.cloak)
  126. # [15:48] * Joins: gitbot (~gitbot@public.cloak)
  127. # [15:48] -gitbot:#testing- [web-platform-tests] Ms2ger pushed 2 new commits to master: https://github.com/w3c/web-platform-tests/compare/86ce2448953d...1384b533acd8
  128. # [15:48] -gitbot:#testing- web-platform-tests/master 092556e Aryeh Gregor: Fix incorrect setAttribute test
  129. # [15:48] -gitbot:#testing- web-platform-tests/master 1384b53 Ms2ger: Merge pull request #842 from ayg/submission/ayg-fix-setattribute-test...
  130. # [15:48] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  131. # [16:03] * Joins: zcorpan (~zcorpan@public.cloak)
  132. # [16:10] * Quits: zcorpan (~zcorpan@public.cloak) (Ping timeout: 180 seconds)
  133. # [16:19] * Quits: plh (plehegar@public.cloak) ("Leaving")
  134. # [16:23] * Joins: gitbot (~gitbot@public.cloak)
  135. # [16:23] -gitbot:#testing- [web-platform-tests] sideshowbarker pushed 1 new commit to submission/sideshowbarker/datetime: https://github.com/w3c/web-platform-tests/commit/9e8a2af4be1ef619f350af1c73b63922d13a94b7
  136. # [16:23] -gitbot:#testing- web-platform-tests/submission/sideshowbarker/datetime 9e8a2af Michael[tm] Smith: Various tweeks.
  137. # [16:23] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  138. # [16:23] <jgraham> You still have to tell critic about squashes because they are still history rewrites
  139. # [16:57] * Joins: Lachy (~Lachy@public.cloak)
  140. # [16:57] * Joins: abarsto (~abarsto@public.cloak)
  141. # [16:57] * abarsto is now known as ArtB
  142. # [17:08] * Quits: MK (kawamori@public.cloak) (Ping timeout: 180 seconds)
  143. # [17:08] * Quits: yankhates_cb (~yankhatescb@public.cloak) (Ping timeout: 180 seconds)
  144. # [17:23] * Quits: darobin (rberjon@public.cloak) (Client closed connection)
  145. # [17:28] * Joins: lukeis (~Adium@public.cloak)
  146. # [17:34] * Joins: anssik (~uid10742@public.cloak)
  147. # [17:42] * Joins: darobin (rberjon@public.cloak)
  148. # [17:45] * Joins: lmclister (~lmclister@public.cloak)
  149. # [17:47] * Joins: rhauck (~Adium@public.cloak)
  150. # [17:59] * Joins: plh (plehegar@public.cloak)
  151. # [18:10] * Quits: ptressel (~chatzilla@public.cloak) (Client closed connection)
  152. # [18:10] * Joins: ptressel (~chatzilla@public.cloak)
  153. # [18:14] * Quits: hober (~ted@public.cloak) (Client closed connection)
  154. # [18:15] * Joins: hober (~ted@public.cloak)
  155. # [18:24] * Parts: plh (plehegar@public.cloak) (Leaving)
  156. # [18:24] * Joins: plh (plehegar@public.cloak)
  157. # [18:24] <plh> so critic is unable to send emails to original PR submitters if they didn't set their account.
  158. # [18:25] <plh> is there anyway we could either use the email address configured in github or ping back the original PR in github?
  159. # [18:26] <plh> we could send a comment back to the PR with a warning or something
  160. # [18:28] * Quits: shepazu (schepers@public.cloak)
  161. # [18:28] * Quits: scott_gonzalez (~scott_gonzalez@public.cloak) (Client closed connection)
  162. # [18:28] * Joins: scott_gonzalez (~scott_gonzalez@public.cloak)
  163. # [18:29] * Joins: shepazu (schepers@public.cloak)
  164. # [18:30] <jgraham> Yeah, it could probably warn in the PR if it can't find the user associated with the push
  165. # [18:33] <plh> that seems the best indeed
  166. # [18:34] <plh> if the critic source code is available somewhere, I'm willing to look into and see if I can submit a change
  167. # [18:34] <jgraham> For clarity, that didn't mean anything like "I am really eager to do the work to implement that warning"
  168. # [18:35] <jgraham> The version we are running is not really
  169. # [18:35] <jgraham> Or, I didn't push for a long while
  170. # [18:35] <plh> :(
  171. # [18:35] <jgraham> I guess that's easier to fix than the issue itself
  172. # [18:35] <plh> that would be nice indeed
  173. # [18:35] <plh> is tehre a repo for critic?
  174. # [18:36] <jgraham> github.com/jensl/critic
  175. # [18:37] <jgraham> Although that might not be the latest versions, and certainly isn't the one we are running
  176. # [18:38] <jgraham> (critic-review.org has the latest version)
  177. # [18:38] <plh> let me know once you get around doing your push, and I'll look into submitting a PR
  178. # [18:39] <plh> if you'd rather have me ping Jens instead, I can certainly do so
  179. # [18:40] * Joins: JohnJansen (~JohnJansen@public.cloak)
  180. # [18:45] <jgraham> Wel jl doesn't have our integration code either because it got stuck in code review (ironically)
  181. # [18:45] <jgraham> Which is my fault agagin
  182. # [18:45] <jgraham> *again
  183. # [19:09] * Quits: Lachy (~Lachy@public.cloak) ("My MacBook Pro has gone to sleep. ZZZzzz…")
  184. # [19:24] <darobin> jgraham: the runner doesn't take long timeout into account, does it?
  185. # [19:25] <darobin> TH knows about that, but the information is not conveyed back
  186. # [19:26] <darobin> I ask because I'm getting lots of failures for /html/infrastructure/urls/resolving-urls/query-encoding/* which actually work
  187. # [19:26] <Ms2ger> darobin, fwiw, those have been failing in Mozilla automation too
  188. # [19:27] <darobin> Ms2ger: completely? because when I run them directly they fail like ~15/148 but in the runner they just timeout
  189. # [19:27] <Ms2ger> Intermittently
  190. # [19:27] <darobin> they do have the timeout=long meta set
  191. # [19:27] <darobin> mmmmmm
  192. # [19:27] <darobin> maybe they need long long :)
  193. # [19:31] <jgraham> darobin: I fixed that test earlier today
  194. # [19:31] <darobin> jgraham: oh, wunderbar
  195. # [19:31] <jgraham> But the runner might not notice timeout=log
  196. # [19:31] <jgraham> *long
  197. # [19:31] <darobin> I don't think it does, at least I can't see where that would happen in the code
  198. # [19:32] <darobin> does TH surface the information?
  199. # [19:34] <jgraham> darobin: It's in the manifest file
  200. # [19:34] <darobin> aha!
  201. # [19:34] <darobin> indeed
  202. # [19:34] <jgraham> So it shouldn't be too hard to add
  203. # [19:35] <darobin> long means twice the timeout?
  204. # [19:35] <jgraham> No, it means 60s rather than 10s
  205. # [20:14] * Joins: gitbot (~gitbot@public.cloak)
  206. # [20:14] -gitbot:#testing- [web-platform-tests] sideshowbarker pushed 1 new commit to master: https://github.com/w3c/web-platform-tests/commit/400fb26acf8df15cb9976212d9331aa6ab97f76b
  207. # [20:14] -gitbot:#testing- web-platform-tests/master 400fb26 Michael[tm] Smith: cc Checkpoint the messages.json file.
  208. # [20:14] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  209. # [20:19] * Joins: gitbot (~gitbot@public.cloak)
  210. # [20:19] -gitbot:#testing- [web-platform-tests] darobin created dupe-button (+1 new commit): https://github.com/w3c/web-platform-tests/commit/216c102d7d1a
  211. # [20:19] -gitbot:#testing- web-platform-tests/dupe-button 216c102 Robin Berjon: duplicate entry for button
  212. # [20:19] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  213. # [20:27] * Quits: rhauck (~Adium@public.cloak) ("Leaving.")
  214. # [20:54] * Joins: rhauck (~Adium@public.cloak)
  215. # [20:57] * Quits: ArtB (~abarsto@public.cloak) ("Leaving.")
  216. # [21:22] <astearns> FYI: eBay/PayPal are planning a TestTWF event on Content Security Policy in Portland, OR in August
  217. # [21:29] * Quits: rhauck (~Adium@public.cloak) ("Leaving.")
  218. # [22:01] * Joins: abarsto (~abarsto@public.cloak)
  219. # [22:01] * abarsto is now known as ArtB
  220. # [22:36] <AutomatedTester> ooo
  221. # [22:45] <Ms2ger> I long for a testtwf on ssl
  222. # [22:46] <AutomatedTester> Ms2ger: organise it
  223. # [22:46] <AutomatedTester> Ms2ger: how about a Rust workweek to rewrite OpenSSL in Rust?
  224. # [22:47] <Ms2ger> Start from NSS, I think :)
  225. # [22:47] <Ms2ger> But I do think that's a project we would be wise to invest in
  226. # [22:48] <AutomatedTester> Ms2ger: sounds like a good intern project
  227. # [22:49] <Ms2ger> That's actually not such an insane idea
  228. # [22:52] <AutomatedTester> Ms2ger: suggest it then
  229. # [22:53] <Ms2ger> But we should get that testsuite written first
  230. # [22:54] <AutomatedTester> meh... tests... meh
  231. # [22:55] <AutomatedTester> Ms2ger: http://w3cmemes.tumblr.com/image/82129869838
  232. # [22:57] <hober> and http://w3cmemes.tumblr.com/post/82207014741/
  233. # [22:57] <AutomatedTester> hober: missed that one
  234. # [22:59] <Ms2ger> hober, I wish
  235. # [23:02] * Joins: MK (kawamori@public.cloak)
  236. # [23:09] * Joins: zcorpan (~zcorpan@public.cloak)
  237. # [23:13] * Quits: MK (kawamori@public.cloak) (Ping timeout: 180 seconds)
  238. # [23:13] * Joins: rhauck (~Adium@public.cloak)
  239. # [23:57] * Quits: anssik (~uid10742@public.cloak) ("Connection closed for inactivity")
  240. # Session Close: Thu Apr 10 00:00:00 2014

The end :)