[06:50:57] The latest massmessage about newsletters are missing timestamp, which means that it will be escaping auto archiving bots. [07:27:54] Och [07:28:06] My bad, sorry [07:28:16] I'll fix this manually [09:49:39] fixed, thanks @Jan_ainali for noticing! [12:18:36] " FYI, there’..." <- I see, thanks for the information [12:19:24] I made several commits for one review. I would like to squash the commits. [12:19:24] Will the gerrit tickets associated with the previous commits be closed? [14:06:26] not automatically, you can "abandon" them in the Gerrit UI [14:24:29] " not automatic..." <- Good to know, thanks! :) [14:25:31] Another question: if I want to push something in review, should I create the associated task first? Or can I open a review without? [17:18:23] florent: it's always good form, but sometimes skipped if the change is fixing a small bug that doesn't need its own task. [17:19:14] I see! [17:19:19] Thanks :) [18:08:15] (I think I will have loads of questions again, sorry for that ^^') [18:09:45] We love questions! [18:10:55] I see babel (`"@babel/core", "@babel/preset-env" and "babel-core") devDependency declared in our package.json. For what do we use it? [18:11:42] (I don't see much reference of babel, except in package.json, package-lock.json and in jest config in which we tell not to use babel) [18:36:34] florent: if you're referring to the Wikilambda extension, I think babel is a dependency for the Ace code editor, which is bundled with the extension [18:51:47] ori: Yes, indeed. It is used by a worker-coffee.js module in ACE. I wonder how the babel dependency and this file interact each other. [18:52:46] if that's required by ACE, I wonder why it is a `devDependency` and not a pure `dependency` (so it can be fetched in production mode) [19:00:42] And it seems like the babel dependency was added first for the introduction of frontend unit tests: https://gerrit.wikimedia.org/r/c/mediawiki/extensions/WikiLambda/+/676678/ [19:03:49] I'm not sure. resources/lib/ace/src/worker-coffee.js contains an inlined, minified package.json file for Ace, which lists Babel as a devDependency, so possibly it is only needed by Ace in some development mode. [19:05:48] ¯\_(ツ)_/¯ maybe ^^' [19:07:04] Also if I remove the babel dependency, the jest unit tests work