[09:01:40] who's around today? [09:02:33] I am [09:02:37] haha [09:02:43] Krinkle|detached jzaefferer : https://github.com/jquery/infrastructure/issues/233 ? [09:02:45] just a tab away! [09:03:05] https://docs.google.com/document/d/1su2ZTFZbnp_DD-LGZleuEudTFRJT_g4zf7UfGOMQvco/edit [09:03:15] m_gol looked like he was around earlier [09:03:23] markelog too maybe [09:03:35] present [09:03:38] hey-hey [09:03:41] it's a holiday here in the US so some people may not be around [09:03:58] we're getting to be worse than the FRENCH [09:04:01] DaveMethvin: where's the usual long list of nicknames to recall all lost souls? ;) [09:04:02] :) [09:04:38] i figured i'd let sleeping dogs lie since it's a holiday [09:05:22] so, i'd love to do a release but it sure would be nice to have some test to prove everything is working [09:05:54] it was a pretty frustrating incident with npm [09:06:21] should have spend more time checking it [09:06:33] although i never encounter such issue before [09:07:03] yeah i dunno what was up [09:07:13] at least we know to keep an eye on it [09:07:20] and it's easy to fix when it does get broken [09:07:23] so we can figure it out [09:08:05] so swarm is still "down"? [09:08:06] current npm should be fine so we just should make sure we have current ones [09:08:20] good call [09:08:24] worse than down, it's really borked [09:08:25] so jenkins isn't updating the git repo before building [09:08:32] thats the broken part i think [09:08:33] current node isn't always enough as npm updates faster [09:08:38] im unaware of other broken parts [09:08:41] if there are any [09:08:49] so why isn't it updating? [09:08:59] but as far as WHY jenkins isn't updating anymore, I'm assuming this goes back to Krinkle updating jenkins plugins 4 or 5 days ago [09:09:18] I tried a few things, but everything I tried just broke [09:09:33] can we revert? [09:09:53] I don't know - I don't keep a backup of the jenkins plugins so [09:10:31] https://github.com/jquery/infrastructure/issues/233 has more [09:11:13] ok, i'll hope it's fixed and we can do a final later this week [09:11:45] I'll be traveling next week, still could do a release tho [09:11:52] we should have that running before releasing the final [09:12:06] agreed [09:12:29] is there anything that has to land before final? i'd prefer to say "no" unless it's critical [09:12:44] "no" from me [09:12:46] better to do a .1 release for bug fixes [09:13:03] DaveMethvin: not really tho http://bugs.jquery.com/ticket/14634 is a new one on my radar [09:13:09] ok so nobody land anything this week [09:13:18] don't want to land this pre final tho [09:13:33] in the meantime we can continue to work on tix and submit pull requests [09:13:44] what about http://bugs.jquery.com/ticket/14680? [09:14:09] i think that's a .1 as well [09:14:47] honestly we should just try...catch it [09:14:47] there are a lot of bugs in IE11 it seems [09:15:01] event on document was clever [09:15:06] i am hoping they'll do some slipstreams [09:15:21] but it getting to complex i guess [09:15:25] yeah so the github blog [09:15:52] basically just grouping all the delegated events into a single query [09:16:01] rather than checking each individually [09:16:19] works fine for their use case but either doesn't help or breaks for a lot of others [09:16:23] no oldIE support either [09:16:42] only works on document because qSA can't do rooted queries [09:16:44] and it's big [09:16:47] very big [09:17:04] but i think bringing back quickIs from 1.7.2 would be a compromise [09:17:17] why did we remove it? [09:17:17] why was it removed? [09:17:21] lol [09:17:23] -) [09:17:26] space saving [09:17:31] it's not too big tho [09:17:32] DaveMethvin: now you have to answer twice [09:17:37] space saving :) [09:17:41] )) [09:17:45] there's an echo in here! [09:18:09] also i thought of another optimization that could do that would improve perf for the github case [09:18:28] how much did we save after it was removed? [09:18:42] can't recall [09:18:52] but we can see what it costs when i put it back [09:18:56] :) [09:19:09] :) [09:19:09] i don't think it's too much [09:19:33] and we know that code works since we had it in previously [09:19:51] the github code is using a bunch of ES5 stuff and would be a pain to back-port to 1x [09:19:59] +1 from me, i like perf improvments [09:20:25] attaching 150 delegated click events to document makes me sad [09:20:49] but their use case is that they have a server side framework and are treating html/js/css like machine language [09:20:52] DaveMethvin: what another optimization did you have in mind? [09:21:20] in a descendant or child selector, the rightmost thing has to match or it's not worth doing a more expensive check [09:21:28] i have seen something like this [09:21:54] it's a minor change to the regex to capture a leading space/> [09:21:55] i could be common for the front-end frameworks [09:22:21] i don't think perf gets bad until you get 100 or more events of the same type [09:22:25] so it's not THAT common [09:22:35] which explains why nobody complained about it before [09:22:42] or maybe they didn't pin it on jQuery :) [09:23:04] but anyway i think we can bring it down to reasonable levels [09:23:14] i was going to write a blog post about it [09:23:19] in the test case i have seen they (or we) used a monkey-patch, just like folks from github [09:23:24] I guess some MVC frameworks might dispatch a lot of events [09:23:45] esp. those with 2-way data binding [09:23:52] yeah, if you did it with something like "scroll" or "mouseover" then you are really screwed [09:24:06] but for low-freq events you'd need like 100 or more [09:24:22] before you'd see a problem today i'd think [09:24:23] although dev that do something like this probably know about issues like that [09:24:45] you would hope so [09:24:51] the github guys are sharp [09:25:02] and the patch they wrote is really good for their situation [09:25:10] but it doesn't handle anything but document [09:25:14] I'm surprised it's not in CoffeScript :P [09:25:18] ) [09:25:20] given their recent tendencies [09:25:30] doens't work on oldIE, doesn't deal with namespaces, etc [09:25:58] okay, where are we? api.jquery.com [09:26:07] m_gol saw some of this discusson [09:26:18] we should really get rid of Migrate there [09:26:21] scott_gonzalez said he'd go through the pages and see what plugins are involved [09:26:33] i'll be glad to help there [09:26:44] once we have that, we can solve it in a case-by-case basis [09:26:49] yeah i doubt it's too hard [09:26:57] and we can push the fixes upstream if we need to [09:27:08] and as I said, either the plugin is dead and forking doesn't really introduce overhead for us [09:27:16] or it's alive so it should accept compatibility patches [09:27:22] yeah [09:27:37] ok so m_gol you good with pitching in once scott_gonzalez has the list? [09:27:43] yup [09:27:54] don't forget [09:27:59] it's every site on the wordpress [09:28:02] not just api ;) [09:28:06] yeah [09:28:09] Thanks. I have a feeling that finding the pages will be the hard part, but we'll see. [09:28:18] haha [09:28:45] turn on those Migrate messages and we can track it down for sure [09:29:17] okay, last item i had, 1.12/2.2 discussion for San Diego [09:29:56] I'll send around a document, you can throw ideas in there and we can review what's already in Trac as well [09:30:16] gibson042! [09:30:26] yay [09:30:37] yeah, looks like my IRC penalty box session finally expired :) [09:30:47] just about ready to wrap ... did anyone else have anything? [09:30:53] when do we release final version? [09:31:04] i hope later this week, thu or fri [09:31:16] but it depends on the test infrastructure [09:31:29] okay, if no one minds i would like try again [09:31:31] can someone add IE11? [09:31:34] I don't have proper keys [09:31:42] oh to the test mix? [09:31:46] yup [09:31:59] I only did what I could which is adding the entry to localSettings.json [09:32:07] i'm not sure i do either [09:32:10] now sb must join the swarm with IE11 [09:32:28] as Krinkle pointed out, instructions are here: https://github.com/jquery/infrastructure/wiki/TestSwarm#add-a-browser [09:32:40] well, first we have to make swarm work again [09:32:43] ) [09:32:50] so I just modified userAgents [09:32:54] haha, true [09:33:28] as for publishing [09:33:36] how do we handle beta/latest tags? [09:33:47] it seems `npm publish` by default updates `latest` [09:33:58] npm publish --beta [09:34:01] which can make `1.11.0` latest instead of `2.1.0` which we want [09:34:09] it was like --tag beta i think [09:34:21] doesn't --beta set the beta dist-tag? [09:34:25] right [09:34:33] because that's not what we want [09:34:40] whoops [09:34:57] latest should be 2.x. [09:35:05] yes [09:35:09] We can have a tag for 1.x for browserify uses. [09:35:14] If we decide to do that. [09:35:17] we could publish in order 1.11, 2.1 worst case [09:35:21] yeah, was thinking about that, too [09:35:22] They can also just say "jquery": "1.x" [09:35:37] Which seems like the sane way to do 1.x latest. [09:35:42] right [09:35:54] so does --beta set the beta dist-tag or not? anyone knows? [09:36:21] i think we did --tag beta, it's in the irc log from thursday [09:36:24] `npm publish --tag beta` will update the beta tag, not the latest tag. [09:36:38] so how to make it not update any tags? [09:36:43] bc that's what we want for 1.x [09:37:09] I'm not sure if we can. We should ask in #node.js [09:37:10] when we release 2.1.0, both latest & beta should be 2.1.0, we don't want beta to be 1.11.0 [09:37:19] --tag ignorethistag [09:37:22] haha [09:37:46] --tag notthedroids [09:38:13] anyway that must be a solveable problem, we'd need to think about how we'd want to do that in core since we publish both [09:38:26] really, doing in order of 1.11, 2.1 might be easiest [09:38:34] yes, I don't think having to remember to publish 1.x first is a viable solutoon [09:38:41] we'll forget [09:38:43] i rarely remember anything [09:38:44] I actually don't think I've ever seen someone use "latest" as the version in npm. [09:38:56] or we'll have to republish one of them and we'll forget we now have to reset the tag [09:38:58] neither have i [09:38:58] It's either ranges, *, or specifc. [09:39:00] Or SHA [09:39:05] But I've never seen anyone use latest. [09:39:10] scott_gonzalez: npm install package installs package@latest [09:39:18] ohh... [09:39:19] so yes, a lot of people actually use it implicitely [09:39:27] That makes sense. [09:39:36] whelp, that means we should make it work :) [09:40:07] Let's track this in the jquery-release repo. [09:40:11] I'll open an issue right now. [09:40:13] ok [09:40:31] anything else? [09:40:36] haha, it may turn out `npm publish --tags "1.x"` will be the least painful solution to the problem :P [09:41:16] on markelog's request to add IE11, who can do that? [09:41:30] amm [09:41:36] https://github.com/jquery/jquery-release/issues/29 [09:41:38] m_gol i think? [09:41:45] i saw the ua parser changes fly by [09:41:50] I can if I have the keys [09:41:59] Heh, I wonder if anyone has created a tag that clashes with version ranges. [09:41:59] and if jenkins starts cooperating ;) [09:42:26] who has keys? gnarf i am sure [09:42:26] right... they're specified in the same way so that might be problematic [09:43:56] okay then, i have to run and it sounds like we're done [09:44:09] see you guys in -dev! [09:44:11] thanks