/irc-logs / w3c / #testing / 2015-11-12 / end

Options:

Previous day, Next day

  1. # Session Start: Thu Nov 12 00:00:00 2015
  2. # Session Ident: #testing
  3. # [00:29] * Quits: bhill2 (~bhill2@public.cloak) (Client closed connection)
  4. # [00:55] * Joins: zcorpan (~zcorpan@public.cloak)
  5. # [01:02] * Quits: zcorpan (~zcorpan@public.cloak) (Ping timeout: 180 seconds)
  6. # [01:14] * Joins: bhill2 (~bhill2@public.cloak)
  7. # [01:54] * heycam is now known as heycam|away
  8. # [01:56] * Joins: zcorpan (~zcorpan@public.cloak)
  9. # [02:03] * Quits: zcorpan (~zcorpan@public.cloak) (Ping timeout: 180 seconds)
  10. # [02:12] * Quits: rniwa (~textual@public.cloak) ("My Mac has gone to sleep. ZZZzzz…")
  11. # [09:29] * Joins: dom (dom@public.cloak)
  12. # [10:09] * Joins: Ms2ger (~Ms2ger@public.cloak)
  13. # [10:18] * Joins: MikeSmith (~MikeSmith@public.cloak)
  14. # [10:19] * Quits: MikeSmith (~MikeSmith@public.cloak) ("Lost terminal")
  15. # [10:25] * Joins: MikeSmith (~MikeSmith@public.cloak)
  16. # [10:46] <jgraham> MikeSmith, Ms2ger: One of you got time to look at my PR to add a list of good first bugs to testtwf.org? It just steals most of the code from jdm so not much to review. It's pretty awesome tho
  17. # [10:46] <Ms2ger> Link?
  18. # [10:56] <tobie> jgraham: it would be cool to add that to the first page
  19. # [10:56] <tobie> s/that/a link to that/
  20. # [10:57] <jgraham> Oh, MikeSmith already reviewed it
  21. # [10:58] <jgraham> tobie: Yes, indeed. How?
  22. # [10:58] <jgraham> Or where, or some other question word
  23. # [10:59] <tobie> jgraham: in index.html
  24. # [10:59] <tobie> would you like me to send a PR?
  25. # [11:00] <tobie> you can then play with it to change it to your convenience
  26. # [11:00] <jgraham> tobie: That would be great
  27. # [11:01] <jgraham> I meant less "where in the source" and more "where in the design"
  28. # [11:02] <tobie> oh, so the start page has these two blocks a the bottom
  29. # [11:02] <tobie> Learn and Contribute
  30. # [11:03] <tobie> You could add a third one: "get started" or something
  31. # [11:06] <jgraham> Yeah, that could work, although I don't typically scroll down that far
  32. # [11:07] <tobie> oh, well, you might want to move the copy around as a second step.
  33. # [11:08] <tobie> (get the marketing stuff lower down)
  34. # [11:11] <jgraham> Sounds reasonable
  35. # [11:11] <jgraham> Are you going to make a PR or shall I?
  36. # [11:14] * Joins: gitbot (~gitbot@public.cloak)
  37. # [11:14] -gitbot:#testing- [testtwf-website-src] tobie created tobie-get-started (+1 new commit): https://github.com/w3c/testtwf-website-src/commit/e37290a965f8
  38. # [11:14] -gitbot:#testing- testtwf-website-src/tobie-get-started e37290a Tobie Langel: Add a Get Started block to index page.
  39. # [11:14] * Parts: gitbot (~gitbot@public.cloak)
  40. # [11:15] * Joins: gitbot (~gitbot@public.cloak)
  41. # [11:15] -gitbot:#testing- [testtwf-website-src] tobie opened pull request #29: Add a Get Started block to index page. (master...tobie-get-started) https://github.com/w3c/testtwf-website-src/pull/29
  42. # [11:15] * Parts: gitbot (~gitbot@public.cloak)
  43. # [11:15] <tobie> jgraham: sent your way
  44. # [11:18] * Joins: gitbot (~gitbot@public.cloak)
  45. # [11:18] -gitbot:#testing- [testtwf-website-src] jgraham created travis_upgrade (+1 new commit): https://github.com/w3c/testtwf-website-src/commit/4f52da6220fb
  46. # [11:18] -gitbot:#testing- testtwf-website-src/travis_upgrade 4f52da6 James Graham: Use new travis infrastructure
  47. # [11:18] * Parts: gitbot (~gitbot@public.cloak)
  48. # [11:18] <jgraham> tobie: r+ as soon as travis finishes
  49. # [11:18] * jgraham is going to catch a train
  50. # [11:19] <tobie> Travis seems to always fail for testtwf website.
  51. # [11:21] * Joins: gitbot (~gitbot@public.cloak)
  52. # [11:21] -gitbot:#testing- [testtwf-website-src] jgraham opened pull request #30: Use new travis infrastructure (master...travis_upgrade) https://github.com/w3c/testtwf-website-src/pull/30
  53. # [11:21] * Parts: gitbot (~gitbot@public.cloak)
  54. # [11:22] * Joins: gitbot (~gitbot@public.cloak)
  55. # [11:22] -gitbot:#testing- [testtwf-website-src] jgraham closed pull request #29: Add a Get Started block to index page. (master...tobie-get-started) https://github.com/w3c/testtwf-website-src/pull/29
  56. # [11:22] * Parts: gitbot (~gitbot@public.cloak)
  57. # [11:22] <jgraham> Oh, exciting
  58. # [11:22] <jgraham> Should fix that
  59. # [11:22] <jgraham> Now I am late…
  60. # [11:23] <tobie> :(
  61. # [11:54] * Joins: ArtB (~ArtB@public.cloak)
  62. # [12:09] * Quits: ArtB (~ArtB@public.cloak) (Client closed connection)
  63. # [12:09] * Joins: ArtB1 (~ArtB@public.cloak)
  64. # [12:18] <jgraham> Turns out the train was even later
  65. # [12:18] <jgraham> Well I would have made it anyway
  66. # [12:20] * Joins: gitbot (~gitbot@public.cloak)
  67. # [12:20] -gitbot:#testing- [testtwf-website-src] jgraham pushed 1 new commit to master: https://github.com/w3c/testtwf-website-src/commit/4bde3ead814f410ceb0da1c58777ce876efdcbf1
  68. # [12:20] -gitbot:#testing- testtwf-website-src/master 4bde3ea jgraham: Merge pull request #30 from w3c/travis_upgrade...
  69. # [12:20] * Parts: gitbot (~gitbot@public.cloak)
  70. # [12:25] * Joins: gitbot (~gitbot@public.cloak)
  71. # [12:25] -gitbot:#testing- [testtwf-website-src] Ms2ger deleted travis_upgrade at 4f52da6: https://github.com/w3c/testtwf-website-src/commit/4f52da6
  72. # [12:25] * Parts: gitbot (~gitbot@public.cloak)
  73. # [12:26] * Joins: gitbot (~gitbot@public.cloak)
  74. # [12:26] -gitbot:#testing- [testtwf-website-src] jgraham created pr_testing_fixes (+1 new commit): https://github.com/w3c/testtwf-website-src/commit/7c2f8fa3bded
  75. # [12:26] -gitbot:#testing- testtwf-website-src/pr_testing_fixes 7c2f8fa James Graham: Fix testing of PRs
  76. # [12:26] * Parts: gitbot (~gitbot@public.cloak)
  77. # [12:26] <Ms2ger> if ENV['TRAVIS_PULL_REQUEST'].to_s.to_i > 0
  78. # [12:26] <Ms2ger> wat
  79. # [12:27] <jgraham> I assume it first converts the string to a string and then to an integer
  80. # [12:28] <jgraham> But who knows
  81. # [12:30] * Joins: gitbot (~gitbot@public.cloak)
  82. # [12:30] -gitbot:#testing- [testtwf-website-src] jgraham opened pull request #31: Fix testing of PRs (master...pr_testing_fixes) https://github.com/w3c/testtwf-website-src/pull/31
  83. # [12:30] * Parts: gitbot (~gitbot@public.cloak)
  84. # [12:36] * Joins: gitbot (~gitbot@public.cloak)
  85. # [12:36] -gitbot:#testing- [testtwf-website-src] jgraham pushed 1 new commit to pr_testing_fixes: https://github.com/w3c/testtwf-website-src/commit/08789b31abc38b7d9c055cc555a92a50489e15ee
  86. # [12:36] -gitbot:#testing- testtwf-website-src/pr_testing_fixes 08789b3 James Graham: fixup! Fix testing of PRs
  87. # [12:36] * Parts: gitbot (~gitbot@public.cloak)
  88. # [12:48] * Joins: simonstewart (~textual@public.cloak)
  89. # [12:54] * Joins: gitbot (~gitbot@public.cloak)
  90. # [12:54] -gitbot:#testing- [testtwf-website-src] jgraham force-pushed pr_testing_fixes from 08789b3 to eb2824f: https://github.com/w3c/testtwf-website-src/commits/pr_testing_fixes
  91. # [12:54] -gitbot:#testing- testtwf-website-src/pr_testing_fixes eb2824f James Graham: Fix testing of PRs
  92. # [12:54] * Parts: gitbot (~gitbot@public.cloak)
  93. # [12:55] <jgraham> Anyone want to review that? I'm not sure if it's "right" in any sense, but it seems to solve the problem
  94. # [12:56] <AutomatedTester> jgraham: #31?
  95. # [12:57] <jgraham> Yes
  96. # [12:57] * Quits: simonstewart (~textual@public.cloak) (Client closed connection)
  97. # [12:57] * Joins: simonstewart (~textual@public.cloak)
  98. # [12:58] <AutomatedTester> jgraham: looks good to me
  99. # [12:59] * Joins: gitbot (~gitbot@public.cloak)
  100. # [12:59] -gitbot:#testing- [testtwf-website-src] jgraham pushed 1 new commit to master: https://github.com/w3c/testtwf-website-src/commit/c45d50acd787829d33fc02ebc279162409ebcb37
  101. # [12:59] -gitbot:#testing- testtwf-website-src/master c45d50a jgraham: Merge pull request #31 from w3c/pr_testing_fixes...
  102. # [12:59] * Parts: gitbot (~gitbot@public.cloak)
  103. # [13:49] * Joins: plh (plehegar@public.cloak)
  104. # [14:02] * Quits: plh (plehegar@public.cloak) ("Leaving")
  105. # [14:03] * Joins: plh (plehegar@public.cloak)
  106. # [14:24] * Quits: simonstewart (~textual@public.cloak) ("My Mac has gone to sleep. ZZZzzz…")
  107. # [14:28] * ArtB1 is now known as ArtB
  108. # [14:43] * Joins: simonstewart (~textual@public.cloak)
  109. # [14:46] * Quits: simonstewart (~textual@public.cloak) ("My Mac has gone to sleep. ZZZzzz…")
  110. # [14:50] * Quits: plh (plehegar@public.cloak) ("Leaving")
  111. # [14:51] * Joins: plh (plehegar@public.cloak)
  112. # [15:26] * Quits: fwtnb (~fwtnb@public.cloak) (Client closed connection)
  113. # [15:26] * Joins: simonstewart (~textual@public.cloak)
  114. # [15:37] * Quits: simonstewart (~textual@public.cloak) ("Textual IRC Client: www.textualapp.com")
  115. # [15:40] * Joins: simonstewart (~textual@public.cloak)
  116. # [16:00] * Joins: gitbot (~gitbot@public.cloak)
  117. # [16:00] -gitbot:#testing- [web-platform-tests] plehegar created hr-time-worker (+1 new commit): https://github.com/w3c/web-platform-tests/commit/6b03bb8de36e
  118. # [16:00] -gitbot:#testing- web-platform-tests/hr-time-worker 6b03bb8 plehegar: performance.now() in workers exists
  119. # [16:00] * Parts: gitbot (~gitbot@public.cloak)
  120. # [16:01] * Joins: gitbot (~gitbot@public.cloak)
  121. # [16:01] -gitbot:#testing- [web-platform-tests] plehegar opened pull request #2324: performance.now() in workers exists (master...hr-time-worker) https://github.com/w3c/web-platform-tests/pull/2324
  122. # [16:01] * Parts: gitbot (~gitbot@public.cloak)
  123. # [16:02] * terri_offline is now known as terri
  124. # [16:31] * Quits: zcorpan (~zcorpan@public.cloak) (Client closed connection)
  125. # [16:34] * Joins: _23 (~_23@public.cloak)
  126. # [16:35] * Quits: _23 (~_23@public.cloak) ("Page closed")
  127. # [16:50] <Ms2ger> plh, commented
  128. # [16:55] <plh> thank you!
  129. # [16:55] <plh> I didn't know that style indeed
  130. # [17:02] <plh> nifty
  131. # [17:02] <Ms2ger> Thanks :)
  132. # [17:08] * Joins: ShaneM (~ShaneM@public.cloak)
  133. # [17:14] * Joins: gitbot (~gitbot@public.cloak)
  134. # [17:14] -gitbot:#testing- [web-platform-tests] plehegar closed pull request #2324: performance.now() in workers exists (master...hr-time-worker) https://github.com/w3c/web-platform-tests/pull/2324
  135. # [17:14] * Parts: gitbot (~gitbot@public.cloak)
  136. # [17:14] * Joins: gitbot (~gitbot@public.cloak)
  137. # [17:14] -gitbot:#testing- [web-platform-tests] plehegar deleted hr-time-worker at 6b03bb8: https://github.com/w3c/web-platform-tests/commit/6b03bb8
  138. # [17:14] * Parts: gitbot (~gitbot@public.cloak)
  139. # [17:17] * Joins: gitbot (~gitbot@public.cloak)
  140. # [17:17] -gitbot:#testing- [web-platform-tests] plehegar created hr-time-2-worker (+1 new commit): https://github.com/w3c/web-platform-tests/commit/6c357372fbf6
  141. # [17:17] -gitbot:#testing- web-platform-tests/hr-time-2-worker 6c35737 plehegar: performance.now() in workers exists
  142. # [17:17] * Parts: gitbot (~gitbot@public.cloak)
  143. # [17:17] * Joins: gitbot (~gitbot@public.cloak)
  144. # [17:17] -gitbot:#testing- [web-platform-tests] plehegar opened pull request #2325: performance.now() in workers exists (master...hr-time-2-worker) https://github.com/w3c/web-platform-tests/pull/2325
  145. # [17:17] * Parts: gitbot (~gitbot@public.cloak)
  146. # [17:40] <gsnedders> what the hell is in the early history of web-platform-tests?
  147. # [17:40] <gsnedders> we seem to have the whole Web DOM Core spec in there
  148. # [17:41] <Ms2ger> Yep
  149. # [17:41] <Ms2ger> You can blame me for that
  150. # [17:41] <gsnedders> can you tell me how old w-p-t actually is? :P
  151. # [17:42] <gsnedders> I want to say ~2010, given when testharness.js was written and such like?
  152. # [17:44] <Ms2ger> I think we started from the webapps or html hg repo
  153. # [17:44] <Ms2ger> Looks like those repos have been nuked from dvcs.w3.org
  154. # [17:45] <gsnedders> html, I think
  155. # [17:45] <gsnedders> as w-p-t I think only 2013 looking around?
  156. # [17:46] <Ms2ger> https://github.com/w3c/web-platform-tests/pull/1 is 2012
  157. # [17:46] <Ms2ger> So late 2012, it seems
  158. # [17:47] <gsnedders> oh well
  159. # [17:54] * Quits: Ms2ger (~Ms2ger@public.cloak) (Ping timeout: 180 seconds)
  160. # [18:09] * Joins: bhill2 (~bhill2@public.cloak)
  161. # [18:11] * Joins: Ms2ger (~Ms2ger@public.cloak)
  162. # [18:19] * Quits: Ms2ger (~Ms2ger@public.cloak) ("nn")
  163. # [18:33] * Quits: dom (dom@public.cloak) ("")
  164. # [18:41] * Quits: bhill2 (~bhill2@public.cloak) (Client closed connection)
  165. # [18:47] * terri is now known as terri_offline
  166. # [18:53] <jgraham> I would have guessed earlier than that
  167. # [18:53] <jgraham> The first testtwf event was October 2012
  168. # [18:53] <jgraham> *June
  169. # [19:03] * terri_offline is now known as terri
  170. # [19:04] * terri is now known as terri_offline
  171. # [19:11] * Joins: bhill2 (~bhill2@public.cloak)
  172. # [19:53] * Quits: bhill2 (~bhill2@public.cloak) (Client closed connection)
  173. # [19:55] * Quits: simonstewart (~textual@public.cloak) ("Textual IRC Client: www.textualapp.com")
  174. # [20:04] * terri_offline is now known as terri
  175. # [20:06] * Quits: plh (plehegar@public.cloak) ("Leaving")
  176. # [20:20] <gsnedders> If I wanted to write one line on our policy on accepting reviews elsewhere, what would it be? Because I want to counter the argument that "people do stupid stuff". Which tbf has happened with Blink and html5lib-tests, where half the tests they changed became invalid…
  177. # [20:22] <gsnedders> "we're willing to accept reviews done elsewhere publicly (this means the reviews done before landing new/modified tests downstream in Gecko/Blink/Servo suffice for landing them upstream; this too has never actually caused any issue)"?
  178. # [20:27] * Joins: bhill2 (~bhill2@public.cloak)
  179. # [20:27] * Quits: bhill2 (~bhill2@public.cloak) (Client closed connection)
  180. # [20:28] * Joins: bhill2 (~bhill2@public.cloak)
  181. # [20:31] <jgraham> gsnedders: I think I didn't understand the question
  182. # [20:31] <jgraham> The line you wrote sounds fine
  183. # [20:32] * Joins: zcorpan (~zcorpan@public.cloak)
  184. # [20:32] <jgraham> If people start landing problematic amounts of bad stuff due to this policy we will ask them not to, or take away push access
  185. # [20:33] <gsnedders> is anyone actually doing it entirely automated?
  186. # [20:33] <gsnedders> AFAIK no?
  187. # [20:33] <jgraham> What do you mean "entirely automated"? The gecko and servo processes are basically entirely automated when it comes to upstreaming
  188. # [20:34] * Joins: plh (plehegar@public.cloak)
  189. # [20:34] <jgraham> I mean it's not like I re-review all the changes before they get upstreamed or anything
  190. # [20:34] <jgraham> (I am in the loop to check that we don't upstream a backout or something equally silly)
  191. # [20:35] <jgraham> (whih otherwise would happen rather often)
  192. # [20:36] <gsnedders> jgraham: I mean automatically grab changes from say gecko, and push them to wpt
  193. # [20:36] <gsnedders> which AFAIK nobody does anything like that because it'd end with silliness
  194. # [20:37] <jgraham> …
  195. # [20:37] <gsnedders> as you said, I mean
  196. # [20:37] <jgraham> Well that sounds quite like what we do
  197. # [20:37] <gsnedders> well you're still in the loop, so it's not entirely automated
  198. # [20:38] <gsnedders> there's still some manual intervention
  199. # [20:38] <jgraham> Only because I'm not clever enough to figure out how to determine if two non-adjacent pushes are backouts of each other
  200. # [20:39] <jgraham> (well there might be some other cases too, but I can't think of them)
  201. # [20:44] * gsnedders wonders when we'll start seeing html5lib-test-like breakage from reviewed upstream stuff :\
  202. # [20:45] <gsnedders> s/up/down/
  203. # [20:45] <gsnedders> Because downstream breaking tests sucks.
  204. # [20:47] * Quits: ShaneM (~ShaneM@public.cloak) (Ping timeout: 180 seconds)
  205. # [20:47] <jgraham> Well, yeah we might. But the point isn't "when will we see any breakage". Nothing we have or can have prevents any breakage. It's "will we see a point at which the harm from the breakage outweighs the benefits from the contributions"
  206. # [20:48] <jgraham> s/any/all/
  207. # [20:48] <jgraham> Like if one Mozilla contribution in 100 introduces a bug and the other 99 fix bugs or improve coverage, that's clearly a win
  208. # [20:49] <jgraham> The next question is "how do we mitigate that risk" and the answer is to have people running the tests, updating frequently, and providing them with tools that will allow them to notice when a test changes state that wasn't updated
  209. # [20:50] * Quits: zcorpan (~zcorpan@public.cloak) (Client closed connection)
  210. # [20:50] <jgraham> The last one is a little hard because of support files, but we can at least provide tools that allow you to visualise all the changes across an import
  211. # [20:50] <jgraham> That will make spotting these bugs much easier
  212. # [21:05] * Joins: ShaneM (~ShaneM@public.cloak)
  213. # [21:17] * heycam|away is now known as heycam
  214. # [21:22] * Joins: zcorpan (~zcorpan@public.cloak)
  215. # [21:24] * Quits: ArtB (~ArtB@public.cloak) ("Leaving.")
  216. # [21:30] * Quits: zcorpan (~zcorpan@public.cloak) (Client closed connection)
  217. # [21:30] * Joins: zcorpan (~zcorpan@public.cloak)
  218. # [21:37] * Quits: zcorpan (~zcorpan@public.cloak) (Ping timeout: 180 seconds)
  219. # [21:38] * Joins: zcorpan (~zcorpan@public.cloak)
  220. # [21:46] * Quits: zcorpan (~zcorpan@public.cloak) (Client closed connection)
  221. # [21:57] * Quits: bhill2 (~bhill2@public.cloak) (Ping timeout: 180 seconds)
  222. # [21:57] * Joins: rniwa (~textual@public.cloak)
  223. # [22:20] * gsnedders feels like jgraham manages to argue this stuff better than him
  224. # [22:23] * heycam is now known as heycam|away
  225. # [23:00] * heycam|away is now known as heycam
  226. # [23:17] * Quits: plh (plehegar@public.cloak) ("Leaving")
  227. # [23:36] * Joins: bhill2 (~bhill2@public.cloak)
  228. # [23:45] * heycam is now known as heycam|away
  229. # Session Close: Fri Nov 13 00:00:00 2015

Previous day, Next day

Think these logs are useful? Then please donate to show your gratitude (and keep them up, of course). Thanks! — Krijn