Options:
Hide Join/Parts/Quits
Previous day , Next day
# Session Start: Thu Jul 16 00:00:01 2015
# Session Ident: #testing
# [00:10] * Quits: chrisbirk (~textual@public.cloak) ("My MacBook Pro has gone to sleep. ZZZzzz…")
# [00:11] * Joins: Guest (~textual@public.cloak)
# [00:14] * Joins: cvrebert (~cvrebert@public.cloak)
# [00:44] * Quits: Guest (~textual@public.cloak) ("My MacBook Pro has gone to sleep. ZZZzzz…")
# [00:59] * Quits: plh-away (plehegar@public.cloak) ("Leaving")
# [01:14] * Joins: Guest (~textual@public.cloak)
# [01:21] * heycam|away is now known as heycam
# [01:31] <MikeSmith> cvrebert: I may be able to review it today
# [01:32] <MikeSmith> We sorta only have 3.5 or so people doing most reviews
# [01:43] <cvrebert> MikeSmith: I see. Thanks a lot if you end up having time for it.
# [01:44] * Quits: Guest (~textual@public.cloak) ("My MacBook Pro has gone to sleep. ZZZzzz…")
# [02:01] * Quits: cvrebert (~cvrebert@public.cloak) (cvrebert)
# [02:29] * Joins: Guest (~textual@public.cloak)
# [02:32] * terri is now known as terri_offline
# [02:34] * Quits: Guest (~textual@public.cloak) ("My MacBook Pro has gone to sleep. ZZZzzz…")
# [03:09] * Joins: cvrebert (~cvrebert@public.cloak)
# [03:20] <MikeSmith> cvrebert: OK I'm reviewing that PR now
# [03:20] <MikeSmith> cvrebert: you know about Critic?
# [03:20] <MikeSmith> https://critic.hoppipolla.co.uk/r/5471
# [03:20] <cvrebert> thanks
# [03:20] <cvrebert> somewhat. I'm willing to learn
# [03:21] <MikeSmith> well the UX is not the greatest in the world but it's a much more complete review tool than just using github issue comments
# [03:22] <MikeSmith> among other things when I make an issue comment on a particular line, and then you push new commit that addresses that comment, critic is smart enough to make the comment as "addressed"
# [03:23] <MikeSmith> one caveat with it is that if a PR is under review you shouldn't rebase/squash and force push to the review branch
# [03:24] <MikeSmith> because when that happens critic chokes and quits tracking the branch
# [03:24] <cvrebert> simple enough. yeah, I saw the "don't rebase" part of the bot's message
# [03:24] <MikeSmith> ok
# [03:26] <MikeSmith> but what we usually do is that if a review branch ends up with a bunch of commits at the end, then at that point (at the end) the committer does the usual thing of squashing/rebasing the commits and pushing it the the branch so we can then push the "Merge" button and merge it as a single commit
# [03:27] <MikeSmith> so we also often just use "git commit --fixup" for those interim commits
# [03:28] <MikeSmith> another thing to know about is that usually you can find the PR branches mirrored at http://w3c-test.org/submissions/
# [03:28] <cvrebert> right
# [03:29] <MikeSmith> but that's now working this week due to some temporary borkage on the server I need to troubleshoot and fix
# [03:30] <cvrebert> Ganbatte.
# [03:30] <MikeSmith> heh
# [03:30] <MikeSmith> denis: btw, https://github.com/w3c/web-platform-tests/settings/hooks/1932908 all coming back 500 right now. If you a few minutes to look at it that would be great
# [03:36] <MikeSmith> cvrebert: this is just a nit but we usually don't use self-closing tag syntax for void elements in HTML source of our test files if they're meant to be served as text/html (which the vast majority are)
# [03:37] <MikeSmith> cvrebert: also what does "zero these out for cross-browser uniformity" mean
# [03:38] <MikeSmith> also is there some currently maintained version of http://www.w3.org/TR/CSS2/visudet.html#min-max-widths
# [03:38] <MikeSmith> because we don't usually reference /TR docs
# [03:39] <cvrebert> MikeSmith: So, you'd prefer unstable http://dev.w3.org/csswg/css2/ ?
# [03:40] * Joins: gitbot (~gitbot@public.cloak)
# [03:40] -gitbot:#testing- [web-platform-tests] hayatoito closed pull request #2004: Remove tests that use obsolete ShadowRoot methods (master...shadowroot-methods) https://github.com/w3c/web-platform-tests/pull/2004
# [03:40] * Parts: gitbot (~gitbot@public.cloak)
# [03:40] * Joins: gitbot (~gitbot@public.cloak)
# [03:40] -gitbot:#testing- [web-platform-tests] hayatoito pushed 2 new commits to master: https://github.com/w3c/web-platform-tests/compare/1d4c7eb1306b...f98c459d74d3
# [03:40] -gitbot:#testing- web-platform-tests/master 4038852 Takayoshi Kochi: Remove tests that use obsolete ShadowRoot methods...
# [03:40] -gitbot:#testing- web-platform-tests/master f98c459 Hayato Ito: Merge pull request #2004 from TakayoshiKochi/shadowroot-methods...
# [03:40] * Parts: gitbot (~gitbot@public.cloak)
# [03:40] * Joins: gitbot (~gitbot@public.cloak)
# [03:40] -gitbot:#testing- [web-platform-tests] hayatoito closed pull request #2003: Use selection object in the same tree. (master...selection) https://github.com/w3c/web-platform-tests/pull/2003
# [03:40] * Parts: gitbot (~gitbot@public.cloak)
# [03:41] <MikeSmith> cvrebert: yeah (except I'd call it "actively maintained" instead of "unstable")
# [03:41] <MikeSmith> will add a review comment for that
# [03:43] <MikeSmith> cvrebert: also btw the really home of all CSS WG drafts is http://drafts.csswg.org/
# [03:44] <MikeSmith> that http://dev.w3.org/csswg stuff is just a hacky rewrite I set up at the request of the CSS WG a long time ago and that I wish they would just quit using and instead publish docs with the real http://drafts.csswg.org/ URLs
# [03:44] <MikeSmith> so anyway http://drafts.csswg.org/css2/
# [03:53] <cvrebert> MikeSmith: Ok. Fixed spec URLs and rewrote that CSS comment for clarity
# [03:55] <MikeSmith> ok
# [03:55] * MikeSmith looks agains
# [03:58] <MikeSmith> heh "No love for XML around here" indeed
# [03:59] <MikeSmith> cvrebert: OK if you can please squash I will merge it
# [04:00] <MikeSmith> "It's not just good, it's good enough!"
# [04:01] <cvrebert> Squashed
# [04:01] <cvrebert> "Hurry up and close it before anyone has a change of heart." LOL.
# [04:13] <cvrebert> MikeSmith: applied remaining Critic feedback and squashed again
# [04:19] <MikeSmith> yeah the critic developer has a pretty good sense of humore
# [04:21] * Joins: gitbot (~gitbot@public.cloak)
# [04:21] -gitbot:#testing- [web-platform-tests] sideshowbarker closed pull request #1969: Test that fieldset's min-width from HTML's suggested default rendering is overridable (master...fieldset-default-min-width-not-important) https://github.com/w3c/web-platform-tests/pull/1969
# [04:21] * Parts: gitbot (~gitbot@public.cloak)
# [04:21] <MikeSmith> cvrebert: thanks
# [04:21] <MikeSmith> will try to respond more timely for your future PRs
# [04:22] <MikeSmith> but a good strategy is to be on here during EU working hours and ping jgraham and Ms2ger
# [04:23] <MikeSmith> cvrebert: I will look at https://github.com/w3c/web-platform-tests/pull/1992 later as well
# [04:23] <cvrebert> Noted. Thanks a lot!
# [04:31] <MikeSmith> cvrebert: on an unrelated note, do you have any idea of github changed its API recently to require authentication to get collaborator lists?
# [04:32] <MikeSmith> I think the reason our http://w3c-test.org/submissions/ stuff is borked right now is because the script that updates it is trying to get https://api.github.com/repos/w3c/web-platform-tests/collaborators?per_page=100&page=1 but it comes back 403
# [04:33] * heycam is now known as heycam|away
# [04:34] <MikeSmith> hmm no the script is actually trying to do auth to get it
# [04:36] <cvrebert> I don't use the collaborators part of the API, so I don't know. GitHub has been messing around with the org-related APIs in the last couple months though; see https://developer.github.com/changes/
# [04:39] * Joins: playjax (~chatzilla@public.cloak)
# [04:43] * Quits: cvrebert (~cvrebert@public.cloak) (cvrebert)
# [04:44] <MikeSmith> nm
# [04:44] <MikeSmith> fixed
# [04:44] <MikeSmith> I had inadvertedly dropped the bot from the team that has push access
# [04:45] <MikeSmith> denis: nm about the sync.py issue; it was my fault and I've fixed it now
# [04:55] * Joins: Guest (~textual@public.cloak)
# [04:58] * Quits: Guest (~textual@public.cloak) ("My MacBook Pro has gone to sleep. ZZZzzz…")
# [05:04] * Quits: ArtB (~ArtB@public.cloak) ("Leaving.")
# [05:12] * Joins: Guest (~textual@public.cloak)
# [05:19] <MikeSmith> botie, inform cvrebert thanks for filing https://www.w3.org/Bugs/Public/show_bug.cgi?id=28954 ( Clarify that suggested default rendering CSS is overridable unless stated otherwise)
# [05:19] <botie> will do
# [05:30] * heycam|away is now known as heycam
# [05:33] * Joins: cvrebert (~cvrebert@public.cloak)
# [05:33] <botie> cvrebert, at 2015-07-16 03:19 UTC, MikeSmith said: thanks for filing https://www.w3.org/Bugs/Public/show_bug.cgi?id=28954 ( Clarify that suggested default rendering CSS is overridable unless stated otherwise)
# [05:36] <cvrebert> you're very welcome
# [05:46] * Quits: birtles (~sid16523@public.cloak) (Ping timeout: 180 seconds)
# [05:47] * Quits: frewsxcv (~sid86219@public.cloak) (Ping timeout: 180 seconds)
# [05:47] * Quits: tobie (~sid5692@public.cloak) (Ping timeout: 180 seconds)
# [05:47] * Quits: timeless (~sid4015@public.cloak) (Ping timeout: 180 seconds)
# [05:48] * Quits: krit (~sid15081@public.cloak) (Ping timeout: 180 seconds)
# [05:48] * Quits: lmclister______ (~sid13822@public.cloak) (Ping timeout: 180 seconds)
# [05:49] * Quits: mkwst (~sid395@public.cloak) (Ping timeout: 180 seconds)
# [05:50] * Quits: scheib (~sid4467@public.cloak) (Ping timeout: 180 seconds)
# [05:50] * Quits: bret (~sid12421@public.cloak) (Client closed connection)
# [05:55] * Quits: hgl (~hgl@public.cloak) (hgl)
# [05:55] * Joins: hgl (~hgl@public.cloak)
# [05:55] * Joins: frewsxcv (~sid86219@public.cloak)
# [05:59] * Joins: tobie (~sid5692@public.cloak)
# [06:00] * Joins: kochi (~kochi@public.cloak)
# [06:01] * Joins: birtles (~sid16523@public.cloak)
# [06:06] * Quits: kochi (~kochi@public.cloak) ("Leaving.")
# [06:07] * Quits: birtles (~sid16523@public.cloak) (Client closed connection)
# [06:07] * Quits: tobie (~sid5692@public.cloak) (Client closed connection)
# [06:07] * Quits: frewsxcv (~sid86219@public.cloak) (Client closed connection)
# [06:08] * Quits: ato (~sid11@public.cloak) (Client closed connection)
# [06:08] * Joins: birtles (~sid16523@public.cloak)
# [06:08] * Joins: kochi (~kochi@public.cloak)
# [06:08] * Joins: ato (~sid11@public.cloak)
# [06:11] * Joins: mkwst (~sid395@public.cloak)
# [06:11] * Joins: scheib (~sid4467@public.cloak)
# [06:13] * Joins: lmclister______ (~sid13822@public.cloak)
# [06:14] * Joins: bret (~sid12421@public.cloak)
# [06:17] * Quits: bret (~sid12421@public.cloak) (Client closed connection)
# [06:17] * Quits: lmclister______ (~sid13822@public.cloak) (Client closed connection)
# [06:17] * Quits: birtles (~sid16523@public.cloak) (Client closed connection)
# [06:17] * Quits: ato (~sid11@public.cloak) (Client closed connection)
# [06:17] * Quits: scheib (~sid4467@public.cloak) (Client closed connection)
# [06:18] * Joins: birtles (~sid16523@public.cloak)
# [06:18] * Joins: ato (~sid11@public.cloak)
# [06:18] * Quits: mkwst (~sid395@public.cloak) (Client closed connection)
# [06:23] * Joins: timeless (~sid4015@public.cloak)
# [06:25] * Joins: gitbot (~gitbot@public.cloak)
# [06:25] -gitbot:#testing- [web-platform-tests] TakayoshiKochi closed pull request #859: Added test for checking if elementFromPoint() method is existing on Shadow Root. (master...shadow-dom-additional) https://github.com/w3c/web-platform-tests/pull/859
# [06:25] * Parts: gitbot (~gitbot@public.cloak)
# [06:27] * Joins: frewsxcv (~sid86219@public.cloak)
# [06:28] * Joins: gitbot (~gitbot@public.cloak)
# [06:28] -gitbot:#testing- [web-platform-tests] TakayoshiKochi opened pull request #2006: Use createShadowRoot() instead of createSR() wrapper. (master...create_shadow_root) https://github.com/w3c/web-platform-tests/pull/2006
# [06:28] * Parts: gitbot (~gitbot@public.cloak)
# [06:28] * Joins: gitbot (~gitbot@public.cloak)
# [06:28] -gitbot:#testing- [web-platform-tests] TakayoshiKochi closed pull request #2006: Use createShadowRoot() instead of createSR() wrapper. (master...create_shadow_root) https://github.com/w3c/web-platform-tests/pull/2006
# [06:28] * Parts: gitbot (~gitbot@public.cloak)
# [06:28] * Joins: gitbot (~gitbot@public.cloak)
# [06:28] -gitbot:#testing- [web-platform-tests] TakayoshiKochi pushed 2 new commits to master: https://github.com/w3c/web-platform-tests/compare/082a6ad6404d...53f31f42769d
# [06:28] -gitbot:#testing- web-platform-tests/master 1351876 Takayoshi Kochi: Use createShadowRoot() instead of createSR() wrapper.
# [06:28] -gitbot:#testing- web-platform-tests/master 53f31f4 Takayoshi Kochi: Merge pull request #2006 from TakayoshiKochi/create_shadow_root...
# [06:28] * Parts: gitbot (~gitbot@public.cloak)
# [06:28] * Joins: bret (~sid12421@public.cloak)
# [06:28] * Joins: lmclister______ (~sid13822@public.cloak)
# [06:28] * Joins: scheib (~sid4467@public.cloak)
# [06:28] * Joins: mkwst (~sid395@public.cloak)
# [06:29] * Quits: Guest (~textual@public.cloak) ("My MacBook Pro has gone to sleep. ZZZzzz…")
# [06:31] * Joins: MK (kawamori@public.cloak)
# [06:36] * Joins: krit (~sid15081@public.cloak)
# [06:46] * Joins: Guest (~textual@public.cloak)
# [06:47] * Joins: tobie (~sid5692@public.cloak)
# [06:52] * Quits: Guest (~textual@public.cloak) (Client closed connection)
# [07:18] * Quits: cvrebert (~cvrebert@public.cloak) (cvrebert)
# [07:25] <MikeSmith> jgraham: so btw the other day this happened:
# [07:25] <MikeSmith> PID USER PR NI VIRT RES SHR S %CPU %MEM TIME+ COMMAND
# [07:25] <MikeSmith> 3359 github 20 0 16.9g 1.8g 1828 S 85.8 46.1 13178:39 python
# [07:25] <MikeSmith> seems that wptserve must be leaking memory pretty seriously
# [07:26] <MikeSmith> can work around it by just periodically stopping and restarting it; e.g., once a week or so
# [07:26] <MikeSmith> or even once a day
# [07:27] <MikeSmith> I guess I should write a cron job to do that
# [07:27] * Joins: MK_ (kawamori@public.cloak)
# [07:27] * Quits: MK_ (kawamori@public.cloak) ("Leaving...")
# [07:31] * Joins: gitbot (~gitbot@public.cloak)
# [07:31] -gitbot:#testing- [web-platform-tests] sideshowbarker closed pull request #1986: Cleaned up the Notifications tests. (master...sideshowbarker/notifications) https://github.com/w3c/web-platform-tests/pull/1986
# [07:31] * Parts: gitbot (~gitbot@public.cloak)
# [07:33] * heycam is now known as heycam|away
# [07:49] * Joins: gitbot (~gitbot@public.cloak)
# [07:49] -gitbot:#testing- [web-platform-tests] sideshowbarker pushed 1 new commit to sideshowbarker/notifications: https://github.com/w3c/web-platform-tests/commit/aa5c1a40931e217c6d2c9e4adb9b0e72094808e5
# [07:49] -gitbot:#testing- web-platform-tests/sideshowbarker/notifications aa5c1a4 Michael[tm] Smith: fixup! Cleaned up the Notifications tests.
# [07:49] * Quits: MK (kawamori@public.cloak) ("Leaving...")
# [07:49] * Parts: gitbot (~gitbot@public.cloak)
# [07:53] * Joins: gitbot (~gitbot@public.cloak)
# [07:53] -gitbot:#testing- [web-platform-tests] sideshowbarker closed pull request #1986: Cleaned up the Notifications tests. (master...sideshowbarker/notifications) https://github.com/w3c/web-platform-tests/pull/1986
# [07:53] * Parts: gitbot (~gitbot@public.cloak)
# [07:53] * Joins: gitbot (~gitbot@public.cloak)
# [07:53] -gitbot:#testing- [web-platform-tests] sideshowbarker deleted sideshowbarker/notifications at aa5c1a4: https://github.com/w3c/web-platform-tests/commit/aa5c1a4
# [07:53] * Parts: gitbot (~gitbot@public.cloak)
# [07:54] * Joins: gitbot (~gitbot@public.cloak)
# [07:54] -gitbot:#testing- [web-platform-tests] sideshowbarker created sideshowbarker/notifications (+2 new commits): https://github.com/w3c/web-platform -tests/compare/29e47810e2b0^...a89ea9c50c60
# [07:54] -gitbot:#testing- web-platform-tests/sideshowbarker/notifications 29e4781 Michael[tm] Smith: Cleaned up the Notifications tests.
# [07:54] -gitbot:#testing- web-platform-tests/sideshowbarker/notifications a89ea9c Michael[tm] Smith: Removed existing Notification dir(manual)tests....
# [07:54] * Parts: gitbot (~gitbot@public.cloak)
# [07:54] * Joins: gitbot (~gitbot@public.cloak)
# [07:54] -gitbot:#testing- [web-platform-tests] sideshowbarker opened pull request #2007: Cleaned up the Notifications tests. (master...sideshowbarker/notifications) https://github.com/w3c/web-platform-tests/pull/2007
# [07:54] * Parts: gitbot (~gitbot@public.cloak)
# [09:29] * Joins: Ms2ger (~Ms2ger@public.cloak)
# [09:29] * Quits: playjax (~chatzilla@public.cloak) (Ping timeout: 180 seconds)
# [10:05] * Joins: dom (dom@public.cloak)
# [10:07] * heycam|away is now known as heycam
# [10:29] * heycam is now known as heycam|away
# [10:39] * Joins: gitbot (~gitbot@public.cloak)
# [10:39] -gitbot:#testing- [web-platform-tests] zqzhang closed pull request #2007: Cleaned up the Notifications tests. (master...sideshowbarker/notifications) https://github.com/w3c/web-platform-tests/pull/2007
# [10:39] * Parts: gitbot (~gitbot@public.cloak)
# [10:39] * Joins: gitbot (~gitbot@public.cloak)
# [10:39] -gitbot:#testing- [web-platform-tests] zqzhang deleted sideshowbarker/notifications at a89ea9c: https://github.com/w3c/web-platform-tests/commit/a89ea9c
# [10:39] * Parts: gitbot (~gitbot@public.cloak)
# [10:48] <jgraham> MikeSmith:
# [10:48] <jgraham> oh, that's bad
# [10:48] <Ms2ger> ?
# [10:49] <MikeSmith> jgraham: yeah but it's not disasterous
# [10:49] <jgraham> wptrunner taking up gigabytes of RAM
# [10:49] <MikeSmith> and we have a workaround: just restart now and then
# [10:49] <MikeSmith> PID USER PR NI VIRT RES SHR S %CPU %MEM TIME+ COMMAND
# [10:49] <MikeSmith> 3359 github 20 0 16.9g 1.8g 1828 S 85.8 46.1 13178:39 python
# [10:50] <MikeSmith> one the one had I admire it, in that if you're gonna leak memory, then you should go ahead and do it right and leak tons of it
# [10:50] <MikeSmith> like a boss
# [10:52] <Ms2ger> Quick, run the reflection tests a few more times
# [11:01] <MikeSmith> hahah
# [11:06] <MikeSmith> Ms2ger: condolences on becoming a full-time wage slave, btw
# [11:06] <Ms2ger> Thank you :)
# [11:07] <Ms2ger> Could be worse, I could be at W3C ;
# [11:07] <Ms2ger> )
# [11:15] <MikeSmith> haha
# [11:16] <MikeSmith> http://w3cmemes.tumblr.com/post/124023684447/it-can-always-be-worse
# [11:17] <MikeSmith> w3cmemes is kinda mean sometimes
# [11:17] <MikeSmith> hmm, kinda like Ms2ger
# [11:18] <MikeSmith> and "Ms2ger" is an acronym for "W3CMemes"
# [11:18] <MikeSmith> just with one less letter, and some other letters change
# [11:54] * Joins: darobin (rberjon@public.cloak)
# [13:01] * Quits: darobin (rberjon@public.cloak) (Client closed connection)
# [13:15] * Joins: playjax (~chatzilla@public.cloak)
# [13:23] * Joins: ArtB (~ArtB@public.cloak)
# [13:31] * Joins: darobin (rberjon@public.cloak)
# [13:41] * Quits: darobin (rberjon@public.cloak) (Ping timeout: 180 seconds)
# [14:12] * Quits: playjax (~chatzilla@public.cloak) (Client closed connection)
# [14:14] * Joins: darobin (rberjon@public.cloak)
# [14:14] * Joins: plh (plehegar@public.cloak)
# [14:40] * Quits: plh (plehegar@public.cloak) ("Leaving")
# [14:41] * Joins: plh (plehegar@public.cloak)
# [14:42] * Quits: Ms2ger (~Ms2ger@public.cloak) (Ping timeout: 180 seconds)
# [16:58] * Quits: darobin (rberjon@public.cloak) (Client closed connection)
# [17:28] * Joins: darobin (rberjon@public.cloak)
# [17:36] * Quits: darobin (rberjon@public.cloak) (Ping timeout: 180 seconds)
# [17:37] * Joins: darobin (rberjon@public.cloak)
# [17:44] * Quits: darobin (rberjon@public.cloak) (Ping timeout: 180 seconds)
# [17:44] * Joins: darobin (rberjon@public.cloak)
# [17:59] * terri_offline is now known as terri
# [18:01] * Quits: dom (dom@public.cloak) ("")
# [18:19] * Quits: darobin (rberjon@public.cloak) (Ping timeout: 180 seconds)
# [19:29] * Joins: darobin (rberjon@public.cloak)
# [19:36] * Quits: darobin (rberjon@public.cloak) (Ping timeout: 180 seconds)
# [20:27] * Joins: Ms2ger (~Ms2ger@public.cloak)
# [20:35] * Quits: rego (~smuxi@public.cloak) (Ping timeout: 180 seconds)
# [22:02] * Joins: rego (~smuxi@public.cloak)
# [22:30] * heycam|away is now known as heycam
# [23:05] * Quits: Ms2ger (~Ms2ger@public.cloak) (Ping timeout: 180 seconds)
# Session Close: Fri Jul 17 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