Options:
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