[Wikitech-l] MediaWiki-CodeSniffer 14.1.0 released

2017-10-23 Thread Kunal Mehta
Hello! MediaWiki-CodeSniffer 14.1.0 is now available for use in your MediaWiki extensions and other projects. This release features improves to the documentation-related sniffs and new upstream releases. 14.0.0 was tagged, but 14.1.0 should be used as it includes a newer upstream version. Here's t

[Wikitech-l] MediaWiki npm jenkins jobs moved over to new docker system

2017-10-30 Thread Kunal Mehta
Hi, I just moved the npm jobs in the MediaWiki queue over to the new, faster, docker based CI system. Instead of requiring building a fresh VM for each 20-30 second job, it uses a docker container. I'm expecting this to make a noticeable improvement in CI response times. For the most part everyth

[Wikitech-l] Migrating away from nose (was 2017-11-01 Scrum of Scrums meeting notes)

2017-11-01 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hi, On 11/01/2017 10:53 AM, Grace Gellerman wrote: > === Scoring Platform === ** Converted our tests from nosetests to > pytest (Anyone else making the transition?) *** nosetests is > under-maintained *** python community is moving towards pytest **

[Wikitech-l] Try out the commit message validator

2017-11-06 Thread Kunal Mehta
Hi! tl;dr Commit messages are pretty important - they make it easy to figure out what a change does without reading the entire diff. And because of that, we have some guidelines[1] on how to format your commit message in the standard way.

Re: [Wikitech-l] TechCom Radar, 2017-11-08

2017-11-15 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hi, On 11/12/2017 01:17 PM, Daniel Kinzler wrote: > As always, the discussion will take place in the IRC channel > #wikimedia-office on Wednesday - however, this time, it will be two > hours earlier! Please join us at 12 noon PST / 21:00 CET / 20:0

[Wikitech-l] Restricting binaries that MediaWiki shells out to

2017-12-04 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hi, For a while now, Wikimedia has been restricting the different binaries that we shell out to (mostly image handling things) with firejail[1]. This was a manual process by writing wrapper scripts that invoked firejail, and pointing MediaWiki to us

[Wikitech-l] RfC: PlatformSettings.php

2017-12-04 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hi, I've written an RfC to introduce a standardized and recommended way for re-distributors and packagers of MediaWiki to be able to tune DefaultSettings.php as appropriate. You can read more details on-wiki[1], and discuss it on Phabricator[2]. [

Re: [Wikitech-l] +2 nomination for WMDE-Fisch in mediawiki/*

2017-12-13 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 I just closed the request as successful - congratulations Fisch! - -- Legoktm -BEGIN PGP SIGNATURE- iQJLBAEBCgA1FiEE+h6fmkHn9DUCyl1jUvyOe+23/KIFAloxgjoXHGxlZ29rdG1A bWVtYmVyLmZzZi5vcmcACgkQUvyOe+23/KI4hRAAl71AhawYYxoyVoh1p0NTcZ3P DSALUlPAtM

[Wikitech-l] Using a PSR-4 autoloader in MediaWiki

2017-12-14 Thread Kunal Mehta
Hi, You can now use a PSR-4 autoloader[1] in MediaWiki core and via extension.json. If you're not familiar with PSR-4, it's a standard which tells autoloaders that if a class is named Foo\Bar\Baz, it can be found at $basePath/Bar/Baz.php. This means that we don't need to keep a mapping of classes

[Wikitech-l] Announcing MediaWiki code search

2017-12-20 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hi, MediaWiki code search is a fully free software tool that lets you easily search through all of MediaWiki core, extensions, and skins that are hosted on Gerrit. You can limit your search to specific repositories, or types of repositories too. Reg

[Wikitech-l] MediaWikiTestCase now validates @cover tags

2017-12-29 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hi, MediaWikiTestCase will now verify[1] that @covers tags are sane when running normal PHPUnit tests. Previously PHPUnit would only verify that when it ran the twice daily coverage job, and fail if even a single one was wrong. This also affects ex

[Wikitech-l] MediaWiki-CodeSniffer 15.0.0 released

2017-12-29 Thread Kunal Mehta
lease notes[1]. The full MediaWiki-CodeSniffer changelog since 14.1.0: * Add sniff for using is_int over is_integer (Kunal Mehta) * Allow _ in unit test method names (Gergő Tisza) * Check function definitions for the same variable name (Kunal Mehta) * Fix handling of alternative

[Wikitech-l] Thank you 2017 code reviewers

2018-01-07 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hi, Nemo generated a list of the top +2'ers for MediaWiki core and MediaWiki extensions in 2017: Thank you very much for all of the code review that you do :-) Special shout out to Timo with 365 +2

[Wikitech-l] Generating PHP test coverage reports for extensions

2018-01-23 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hi, CI now supports generating PHP test coverage reports for extensions after a commit is merged. It's already running for some extensions: Hopefully these reports will encourage people to increase tes

Re: [Wikitech-l] +2 nomination for WMDE-leszek in mediawiki/*

2018-01-31 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 I've closed the request as successful. Congrats Leszek! - -- Legoktm On 01/23/2018 10:36 AM, Addshore wrote: > Hi, > > I've filed , nominating > WMDE-leszek for +2 in mediawiki/ repos. > > -- Addshore _

[Wikitech-l] MediaWiki codesearch updates, co-maintainers wanted

2018-01-31 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hi, A few updates on codesearch.wmflabs.org: * The OOUI git repository is now indexed * The operations/dns and operations/mediawiki-config git repositories are now indexed * Should properly restart automatically after rebooting the instance * Bette

[Wikitech-l] Test coverage reports for patchsets

2018-02-02 Thread Kunal Mehta
Hi, I've been working on a tool, phpunit-patch-coverage[0], that tries to provide test coverage reports for patches before they are merged[1]. Since it takes 2-3 hours to do a full coverage report for MediaWiki core, the tool figures out what files were modified, and runs those tests. For now I'v

[Wikitech-l] MediaWiki-CodeSniffer 16.0.0 released

2018-02-14 Thread Kunal Mehta
(Kunal Mehta) * Add sniff to use namespaced PHPUnit\Framework\TestCase (Kunal Mehta) * Avoid assignment in return statements (Umherirrender) * Be aware of extension MediaWiki compatibility (Kunal Mehta) * Detect variadic arguments in function comments (Umherirrender) * Disallow PHP 7.2+ `object` type

Re: [Wikitech-l] MediaWiki-CodeSniffer 16.0.0 released

2018-02-24 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hi everyone, I just released 16.0.1 to fix a bug[1] that was causing PHPCS to error out in specific cases. Thank you to Umherirrender for both reporting and fixing the issue :-) A few other minor refactoring changes are included: * Fix SpaceBeforeC

[Wikitech-l] codesearch might have returned incorrect results in the past 2ish days

2018-03-01 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hi, For about the past two days codesearch.wmflabs.org ran out of disk space[1] and has been not searching some repositories[2]. This has been fixed now, thanks to bd808 and others in #wikimedia-cloud for their help :) People who were looking for t

[Wikitech-l] Upcoming CI changes for PHP 7

2018-03-05 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hi, We're planning to enable tests against PHP 7.0 as a voting job for MediaWiki core, and all MediaWiki extensions and skins that currently have one for PHP 5.5 and HHVM. Tests have run in non-voting mode against nearly all extensions, most of whi

Re: [Wikitech-l] Upcoming CI changes for PHP 7

2018-03-08 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hi, On 03/05/2018 10:54 PM, Kunal Mehta wrote: > We're planning to enable tests against PHP 7.0 as a voting job for > MediaWiki core, and all MediaWiki extensions and skins that > currently have one for PHP 5.5 and HHVM. This ha

Re: [Wikitech-l] Example .spec file for packaging Mediawiki with RPM?

2018-03-26 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hi, On 03/26/2018 03:02 PM, Hogan, Michael C wrote: > Anyone know of a Mediawiki RPM .spec file posted online? I've only > been able to find pre-packaged RPMs. Thank you! The .spec files that Fedora uses are available at

Re: [Wikitech-l] Example .spec file for packaging Mediawiki with RPM

2018-03-28 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hi, On 03/28/2018 01:46 PM, Hogan, Michael C wrote: > Kunal, thank you for the reference to the Fedora .spec files. > Anyone know if there's a technical / policy reason for the Fedora > EPEL package to be stuck at mediawiki123? > https://mirror.umd

[Wikitech-l] MediaWiki-CodeSniffer 17.0.0 released

2018-03-28 Thread Kunal Mehta
irst and shorten out earlier (Thiemo Kreu z) * Check for use statement with non-compound name (Umherirrender) * Don't require @covers from abstract tests (Max Semenik) * Forbid usage of assert() (Kunal Mehta) * Improve ClassMatchesFilenameSniff::isMaintenanceScript (Umherirrender)

[Wikitech-l] Review and update your extensions for PHPUnit 6

2018-04-07 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hi, tl;dr: Read the "So what do you need to do?" section We're currently in the process of upgrading the MediaWiki PHPUnit version to 6. To avoid an immediate breaking change, we will be doing this transition in two steps. First, we will start runn

[Wikitech-l] Proposal to remove $wgValidateAllHtml and MWTidy::checkErrors() without deprecation

2018-04-10 Thread Kunal Mehta
Hi, As part of the goal to have MediaWiki 1.31 default to using RemexHtml and replacing tidy, we want to remove MWTidy::checkErrors() and $wgValidateAllHtml as part of MediaWiki 1.31, without a deprecation period. MWTidy::checkErrors() basically ran the provided input through tidy, and checked if

Re: [Wikitech-l] Review and update your extensions for PHPUnit 6

2018-04-13 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hi, On 04/07/2018 09:03 PM, Kunal Mehta wrote: > I'd like to start running voting tests against PHPUnit 4 and 6 > next week on Thursday (April 12). This should give us just enough > time to fix any resulting issues before the 1.31 b

Re: [Wikitech-l] Proposal to remove $wgValidateAllHtml and MWTidy::checkErrors() without deprecation

2018-04-13 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hi, On 04/10/2018 05:44 PM, Kunal Mehta wrote: > As part of the goal to have MediaWiki 1.31 default to using > RemexHtml and replacing tidy, we want to remove > MWTidy::checkErrors() and $wgValidateAllHtml as part of MediaWiki > 1.3

Re: [Wikitech-l] "PHP test coverage decreased" - really?

2018-04-26 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hi, On 04/26/2018 12:02 PM, Daniel Kinzler wrote: > Hi all! > > While I think it's great that CI is making sure that test coverage > improves, I'm struggeling with making this work for me. It seems > to me the stats that are used to detect test c

Re: [Wikitech-l] 2018-05-02 Scrum of Scrums meeting notes

2018-05-02 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hi, On 05/02/2018 11:02 AM, Grace Gellerman wrote: > * Services: If someone maintains/knows JobQueue Job code that posts > jobs cross-wiki, please tell us. MassMessage and GlobalUserPage both post jobs across wikis and rely on that functionality. A

Re: [Wikitech-l] PHPUnit 6+ on older versions of MW

2018-05-13 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hi, On 05/12/2018 11:37 PM, Jeroen De Dauw wrote: > I updated one of my extensions to use PHPUnit 6+ and am having > troubles getting the tests to run with older versions of MediaWiki. > In particular, I'd like my tests to run with MW 1.27. I suspec

[Wikitech-l] Provide input on the future direction of MediaWiki's extension management system

2018-05-16 Thread Kunal Mehta
Hi everyone, As part of the MediaWiki Platform Team's goal to develop a roadmap for MediaWiki, we want to solicit input on the future direction of MediaWiki's extension management system. One of MediaWiki's greatest strengths is the diverse extension ecosystem. But navigating that ecosystem and ma

Re: [Wikitech-l] PHPUnit 6+ on older versions of MW

2018-05-16 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hi, On 05/15/2018 11:25 PM, Jeroen De Dauw wrote: > Failing tests: > https://travis-ci.org/JeroenDeDauw/Maps/builds/375344161 > > Code: https://github.com/JeroenDeDauw/Maps OK, so you need to use the PHPUnit4And6Compat trait as explained in[1]. Th

[Wikitech-l] MediaWiki core master now requires PHP 7.0 or HHVM

2018-05-25 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hi, MediaWiki core's master branch now requires a minimum of PHP 7.0 or HHVM[1], as decided upon in the RfC. CI has been updated no longer run PHP 5.5 tests. There are already quite a few patches[2] to remove code that isn't necessary now, or start

[Wikitech-l] MediaWiki-CodeSniffer 19.0.0 and 20.0.0 released

2018-05-25 Thread Kunal Mehta
e for traits with phpunit annotations (Umherirrender ) * Don't allow 'iterable' type hint (Kunal Mehta) * Prevent usage of nullable and void type hints (Kunal Mehta) * Prohibit PHP's vanilla execution (Max Semenik) * Reorganize PHP 7.0 compatibility sniffs into a category (Kun

Re: [Wikitech-l] +2 nomination for Jakob in mediawiki/*

2018-06-04 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hi, I've closed the request as successful. Congrats Jakob! - -- Legoktm On 05/20/2018 10:47 AM, Addshore wrote: > Hi, > > I've filed <*https://phabricator.wikimedia.org/T195220 > *>, nominating Jakob fo

Re: [Wikitech-l] Gerrit as a shared community space

2018-06-09 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hi, CoC.md business aside, I agree with the main thing you've said. Specifically: On 06/09/2018 08:58 AM, Brion Vibber wrote: > I think we should though clarify that code repositories on gerrit > and diffusion are not owned by any one person, but a

Re: [Wikitech-l] Thank you week-end thread

2018-06-09 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hi, Thank you for starting this thread :) On 06/08/2018 12:03 PM, Antoine Musso wrote: > Marco Aurelio, who has been dramatically helping to archive a few > outdated/no more working extensions. The workflow is nice, makes > sure nothing get missed

[Wikitech-l] +2 nomination for MarcoAurelio in mediawiki/*

2018-06-13 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hi, I've filed , nominating MarcoAurelio for +2 access in mediawiki/. - -- Legoktm -BEGIN PGP SIGNATURE- iQIzBAEBCgAdFiEE+h6fmkHn9DUCyl1jUvyOe+23/KIFAlsg1OEACgkQUvyOe+23 /KKneQ/8DzSMb5IkWbLEbSxC4y

Re: [Wikitech-l] Extension registration dependencies towards skins

2018-06-15 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hi, On 06/15/2018 04:42 PM, Alex Winkler wrote: > I've been wondering if it is just the documentation being wrong or > the key being typo'd ever since 1.29, considering the extensions > dependency key is inplural I'd tend to think the key is typo'd

Re: [Wikitech-l] +2 nomination for MarcoAurelio in mediawiki/*

2018-06-20 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 I've closed the request as successful. Congrats to MarcoAurelio for being the first new volunteer +2'er since 2016! - -- Legoktm On 06/13/2018 10:25 AM, Kunal Mehta wrote: > Hi, > > I've filed <https://phabri

Re: [Wikitech-l] [Engineering] Additional European-appropriate MediaWiki train window starting week of July 9th (also SWAT change)

2018-07-11 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hi, On 07/11/2018 12:05 AM, Marko Obrovac wrote: > It seems to me that it will be confusing only to non-US people, > honestly, which are the ones benefiting from this change. When DST > changes are applied, and because they are not applied at the sa

[Wikitech-l] eslint compromised, reset your npm tokens

2018-07-12 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hi, If you ran eslint (JavaScript codestyle linter) recently (it was only compromised for an hour), your npm token might have been compromised (~/.npmrc). To identify if you were compromised, run: $ locate eslint-scope | grep -i "eslint-scope/packa

[Wikitech-l] +2 nominations for mediawiki/*

2018-07-18 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hi, Following comments from last time that we need some more volunteer +2'ers, I've filed multiple nominations for +2 rights in mediawiki/*: * Eranroz: https://phabricator.wikimedia.org/T199866 * Huji: https://phabricator.wikimedia.org/T199867 * Ja

Re: [Wikitech-l] +2 nominations for mediawiki/*

2018-07-25 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hi, I just closed all of the nominations as unanimously successful - congratulations to everyone :) - -- Legoktm On 07/18/2018 12:51 AM, Kunal Mehta wrote: > Hi, > > Following comments from last time that we need some more volunteer

[Wikitech-l] New PHPUnit structure test validating existence of right-* i18n messages

2018-07-28 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hi, For proper display on pages like Special:ListGroupRights, all userrights are supposed to have a corresponding right-{name} i18n message. MediaWiki core has had a PHPUnit test enforcing this for quite a while now, so I converted[1] it into a stru

Re: [Wikitech-l] Proposal for removal without deprecation of MagicWord::clearCache

2018-07-31 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hi, I don't have a strong opinion about the specific function in this case, but I did want to discuss a few things about the policy itself. On 07/30/2018 06:28 AM, Stephan Gambke via Wikitech-l wrote: > I know it is annoying as heck to keep outdate

[Wikitech-l] Postgres and SQLite tests are now voting on mediawiki/core patches

2018-08-01 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hi, MediaWiki core patches will now need to pass tests on postgres and sqlite (in addition to mysql of course) to be merged. For speed, CI won't run postgres/sqlite tests until after the patch has been +2'd (gate-and-submit queue). If you want to ru

Re: [Wikitech-l] My Phabricator account has been disabled

2018-08-11 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hi, On 08/08/2018 03:08 AM, Amir Ladsgroup wrote: > I disabled the account and now I disabled it again. It's part of a > CoC ban. We sent the user an email using the "Email to user" > functionality from mediawiki.org the moment I enforced the ban. >

Re: [Wikitech-l] My Phabricator account has been disabled

2018-08-11 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 On 08/09/2018 04:55 AM, Aryeh Gregor wrote: > The main problem here that needs to be solved is communicating > what the problem was in a manner that is clear to the parties whom > the CoC committee seeks to deter. A one-week ban is not going to > he

Re: [Wikitech-l] ForkableMaintenance: thoughts and help with tests/code coverage

2018-08-12 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hi, On 08/11/2018 06:48 PM, Brion Vibber wrote: > Second, probably related to that I'm seeing a failure in the code > coverage calculations -- it's seeing some increased coverage on the > parent process at least but seems to think it's returning a >

[Wikitech-l] MediaWiki-CodeSniffer 22.0.0 released

2018-09-02 Thread Kunal Mehta
etect nesting of inline ternary statements without parentheses (Kevin Israel) * Disable 'Generic.PHP.DeprecatedFunctions' sniff (Kunal Mehta) * Enable PSR2.Classes.ClassDeclaration (Umherirrender) * Enable Squiz.Functions.FunctionDeclarationArgumentSpacing (Umherirr

[Wikitech-l] Extension test failures on PHP 7.1

2018-10-07 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hi, In preparation for Wikimedia production switching to PHP 7.2, we need to get CI running using 7.2 (and for the rest of the MediaWiki world too!). But before we can do that, we'll need 7.1 to be passing first. The following extensions are current

[Wikitech-l] Debian & Ubuntu packages of MediaWiki 1.31 now available

2018-10-10 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hi, Debian and Ubuntu packages of MediaWiki 1.31 (LTS) are now available. See for more information on how to get them, and basic instructions for installing (hint: it's as simple as `sudo apt i

[Wikitech-l] Extension test failures on PHP 7.2

2018-10-14 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hi, On 10/8/18 7:43 AM, Aryeh Gregor wrote: > On Mon, Oct 8, 2018 at 9:04 AM Kunal Mehta > wrote: >> In preparation for Wikimedia production switching to PHP 7.2, we >> need to get CI running using 7.2 (and for the rest of the

Re: [Wikitech-l] Changes to User Blocking

2018-11-04 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hi, On 11/2/18 12:58 PM, Alex Ezell wrote: > *tl:dr;* On a wiki with the new Partial Blocks enabled (currently > only testwiki), if the code is checking User::isBlocked() to > determine edit rights, it should instead check User::isBlockedFrom( > Tit

Re: [Wikitech-l] Extension test failures on PHP 7.2

2018-11-13 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hi, On 10/14/18 11:39 PM, Kunal Mehta wrote: > I'd like to also make 7.2 voting by the end of the week, but I > realize that might be a little too ambitious with multiple > conferences this week s/week/month/ it seems. As of a few

[Wikitech-l] MediaWiki-CodeSniffer 23.0.0 released

2018-11-13 Thread Kunal Mehta
es in @param (Umherirrender) * Update message to talk about "top level" instead of "file comment" (Thiemo Kreuz) * Upgrade squizlabs/php_codesniffer to 3.3.2 (Kunal Mehta) Thanks, - -- Legoktm -BEGIN PGP SIGNATURE- iQIzBAEBCgAdFiEE+h6fmkHn9DUCyl1jUvyOe+23/KIFAlvr0ekACgkQ

[Wikitech-l] Thank you Tuesday

2018-11-20 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hi, It's been a while since we had a thank you thread, so since it's Tuesday, I'm starting one for the alliteration. If you haven't participated in one of these before, it's simple - just reply to the list thanking someone for what they've done rec

Re: [Wikitech-l] Thank you Tuesday

2018-12-05 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hi, On 12/4/18 7:55 AM, MA wrote: > It's Tuesday (my timezone at least) and this week I'd like to thank > the following people for their work and assistance: Sorry for being a day late, here's my thanks: * BPirkle and Reedy for moving forwards wit

[Wikitech-l] Feedback requested on proposal for extensions to "reserve" blocks of namespaces

2018-12-10 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hi, I'm forwarding along a request for feedback on a proposal started by Dinoguy1000 related to the "Extension default namespaces"[1] page. They've proposed codifying the general practice of having extensions reserve blocks of ten namespace numbers

[Wikitech-l] Changes to CI for Python jobs

2018-12-18 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hi, There are a few changes to the Python `tox` jobs that CI is running: 1. Most importantly, Python 3.5 is now the default version when invoked with `tox`. This can be overridden by using basepython[1]. 2. Multiple Python 3 versions are available:

[Wikitech-l] Thank you Thursday

2018-12-20 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hi, Sorry for missing Tuesday, here's this week's thank you thread :) * Thanks to Faidon for packaging different Python 3 versions for Stretch so Wikimedia CI can test against multiple Python versions seamlessly. * Thanks to Jforrester and Paladox

[Wikitech-l] Thank you Tuesday

2019-01-15 Thread Kunal Mehta
Hi everyone, Following up on Andre's statistics[1] about the people writing/editing the code, I wanted to share some about the people reviewing the code (gathered by Nemo, thanks!). The top +2'ers in MediaWiki core: 582 Timo Tijhof 475 James D. Forrester 406 Kunal Mehta 189 Aaron S

Re: [Wikitech-l] [Engineering] Gerrit now automatically adds reviewers

2019-01-22 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hi, On 1/18/19 6:11 AM, Tyler Cipriani wrote: > In the interim, project-owners are able to opt-in to using the > reviewers-by-blame plugin on a per-project basis on their project > admin page in Gerrit. I'm happy to volunteer any of the projects I

Re: [Wikitech-l] Gerrit now automatically adds reviewers

2019-01-22 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hi, On 1/18/19 1:11 AM, Jaime Crespo wrote: > One member of my team sadly left. Now he is pinged every time I > upload a change, passively aggressive reminding him he used to work > on this. > > Don't get me wrong, I think this is great to get atte

Re: [Wikitech-l] UserMerge extension on Wikimedia sites

2019-01-22 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hi, On 1/22/19 12:24 AM, Amir E. Aharoni wrote: > Yes, it's installed, but it doesn't seem to be actually used. The > English Wikivoyage has a bunch of log entries of its usage, the > last in 2015. Other Wikimedia sites have very few entries, or non

[Wikitech-l] Thank you Tuesday

2019-01-22 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hi, It's still Tuesday in my timezone, so here's this week's thread. Noticed something neat or cool that someone did? Or is someone just being awesome in general? Say thanks! * Thanks to Krinkle and Krenair for cleaning up user JavaScript across Wi

Re: [Wikitech-l] The mw.ext construct in lua modules

2019-01-25 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hi, On 1/24/19 11:33 PM, Thiemo Kreuz wrote: > Is there a question assigned with this long email? Is this a call > for feedback? I think this is probably related to/coming from . - -- Legoktm

Re: [Wikitech-l] IE 6/7 MIME type sniffing checks on uploads - is it time to retire them?

2019-01-28 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hi, On 1/28/19 3:58 PM, Brion Vibber wrote: > Years ago, we added security checks for IE 5/6/7 to work around > IE's mime type sniffing: if you went to view a .png file directly > in IE (as opposed to in an ) the browser would check the first > few

[Wikitech-l] MediaWiki-CodeSniffer 24.0.0 released

2019-02-06 Thread Kunal Mehta
, 19.2.0, which just disabled the problematic Generic.PHP.DeprecatedFunctions sniff. Here's the changelog for 24.0.0: * Whitelist @after and @before phpunit annotations (Umherirrender) * Update PHP_CodeSniffer to 3.4.0 (Kunal Mehta) * Enable new Generic.VersionControl.GitMergeConflict sniff (

Re: [Wikitech-l] [Train] 1.33.0-wmf.18 status update

2019-02-19 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hi, On 2/19/19 6:36 AM, Tyler Cipriani wrote: > I have not yet started the 1.33.0-wmf.18 train; however, at the end > of last week, I noticed some errors that (AFAICT) are regressions > in 1.33.0-wmf.17. In addition to those, I'm not sure whether w

[Wikitech-l] Upgrading phan

2019-02-24 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hi, It's time to work on upgrading phan, the PHP static analysis tool we use! The version of phan we were using was released over two years ago (0.8.0) - we're moving to 1.2.4 that was released less than a week ago. And the CI infrastructure is now

[Wikitech-l] Thank you Thursday

2019-02-28 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hi, It's Thursday :) I wanted to say thank you this week to Zoranzoki21 and Matěj Suchánek for jumping on the phan migration and helping with upgrading extensions to take advantage of newer static analysis tools. - -- Legoktm -BEGIN PGP SIGNAT

[Wikitech-l] MediaWiki-CodeSniffer 25.0.0 released

2019-04-05 Thread Kunal Mehta
rame98) * Streamline PHPDoc comment parsing in UnusedUseStatement sniff (Thiemo Kreuz) * Upgrade PHP_CodeSniffer to 3.4.1 (Kunal Mehta) * Whitelist more phan annotations (Kunal Mehta) [1] https://phabricator.wikimedia.org/T218750 - -- Legoktm

Re: [Wikitech-l] Switching to elections for the Code of Conduct Committee?

2019-06-20 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hi, On 6/18/19 3:24 PM, Yaron Koren wrote: > That's what I'm writing about: I now think that the committee > should be decided via open elections, instead of having the > committee appoint itself. At the moment, this group has a complete > lack of a

[Wikitech-l] Dealing with composer dependencies in early MediaWiki initialization

2019-06-25 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hi, I (with Reedy's help) recently started work on librarizing MediaWiki's IP class into a separate composer package (wikimedia/ip-utils[1]). The main motivation was so that the Parsoid PHP port could use it[2]. However, I ran into an unexpected hi

Re: [Wikitech-l] New errors with composer update

2019-06-28 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hi, On 6/26/19 10:53 PM, James Montalvo wrote: > I'm getting a new error today that was not happening with the > identical config yesterday. Tgr tracked down the root cause and filed to figure out how to

[Wikitech-l] +2 nomination for Daimona in mediawiki/*

2019-07-17 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hi, I've nominated Daimona for +2 in mediawiki/* repositories. Please see . Thanks! - -- Legoktm -BEGIN PGP SIGNATURE- iQIzBAEBCgAdFiEE2MtZ8F27ngU4xIGd8QX4EBsFJpsFAl0u2zMACgkQ8QX4EBsF JpsNRxAAsMO1

Re: [Wikitech-l] Help merge patch (Interwiki)

2019-07-18 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hi, On 7/8/19 10:04 AM, Isarra Yos wrote: > Basically, uh, can anyone else help us with this? Any of this? > Given the nature of the patch we really do kind of need it merged > so it can get translations. I went ahead and +2'd it. As I was reviewin

Re: [Wikitech-l] Dealing with composer dependencies in early MediaWiki initialization

2019-07-18 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hi, On 6/26/19 11:25 PM, Tim Starling wrote: > Interesting that I wrote there: "How about this: let's set > $wgServer in the installer in 1.18, and remove $wgServer > autodetection from DefaultSettings.php a bit later, say in 1.20." > > It was inde

Re: [Wikitech-l] +2 nomination for Daimona in mediawiki/*

2019-07-24 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 > I've nominated Daimona for +2 in mediawiki/* repositories. Please > see . I've closed this as successful. Congrats Daimona! - -- Legoktm -BEGIN PGP SIGNATURE- iQIzBAEBCgAdFiEE2MtZ8F27ngU4xIGd8QX

[Wikitech-l] Thank you Thursday

2019-07-25 Thread Kunal Mehta
Hi, This week I'd like to thank: * Daimona, bawolff, and sbassett for their work on v2.0 of the phan-taint-check-plugin, which has been instrumental in catching real security issues. * James_F for picking up a lot of CI maintenance * MatmaRex for seeing through a patch to allow skins to have custo

Re: [Wikitech-l] TechCom Radar 2020-01-22

2020-01-29 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA256 Hi, On 2020-01-28 05:29, Kate Chapman wrote: > * New RFC under discussion: Allow extensions to register handlers > with factories without the use of global variables. > Just to be clear, this is not inten

Re: [Wikitech-l] List related to repositories which haven't mediawiki/mediawiki-codesniffer

2020-03-16 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA256 Hi, On 2020-03-05 22:46, Zoran Dori wrote: > Hello, can someone to update list > https://phabricator.wikimedia.org/P10500 which contains > repositories which haven't mediawiki/mediawiki-codesniffer. I added this functionality to the PHPCS tool:

[Wikitech-l] CodeReview to be removed from mediawiki.org, dumps available

2020-05-18 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA256 Hi everyone, We're planning to review the CodeReview extension, which was used during the SVN era, from mediawiki.org soon. A static HTML dump of the code review comments is available at and SQL dumps are

Re: [Wikitech-l] CodeReview to be removed from mediawiki.org, dumps available

2020-05-19 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA256 Hi, On 2020-05-18 14:46, Zoran Dori wrote: > Hello, >> We're planning to review the CodeReview extension, which was >> used during the SVN era, from mediawiki.org soon. > > You mean to "remove extension" from mediawiki.org, right? :) Yep, my bad. T

Re: [Wikitech-l] +2 for DannyS712 for mediawiki/*

2020-05-21 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA256 I just closed the request as successful. Congrats Danny! - -- Legoktm On 2020-05-14 14:28, Danny S wrote: > Dear all, > > Per policy[1], I'd like to inform you that I have been nominated[2] > for +2 rights for mediawiki/*. > > Stay safe, -DannyS712

Re: [Wikitech-l] CI build of empty patch fails in Vector repo in 'mediawiki/phan-taint-check-plugin'

2020-06-01 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA256 On 2020-06-01 07:21, Aron Demian wrote: > This has been happening since Saturday. Looking for resolution. See . It's currently being fixed by rolling out a new version of the phan-taint-check plugin. - --

Re: [Wikitech-l] Gerrit upgrade on Saturday, 27th of June

2020-06-21 Thread Kunal Mehta
Hi, On 2020-06-20 14:58, Christian Aistleitner via Wikitech-l wrote: > It holds a copy of our Gerrit data as of 15th of June, and it runs > the new Gerrit v3.2.2. We'll keep tuning things there during this > week, so it might get an occasional restart. But look around and > test the new UI, the HT

[Wikitech-l] npm and security (was: Re: TechCom Radar 2020-07-01)

2020-07-06 Thread Kunal Mehta
Hi, On 2020-07-02 02:41, Aron Manning wrote: > Recently, I haven't seen the security question of npm being discussed on > https://phabricator.wikimedia.org/T199004 > and thought that question was not on topic. If that's not the case: is > there another discussion I'm not aware of? There's been s

[Wikitech-l] Proposed breaking change to Command::restrict()

2020-07-08 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA256 Hi, I'm proposing a breaking change to MediaWiki\Shell\Command::restrict(). I made a mistake in my original implementation of this function that I believe we're better off fixing. For background on what shell restrictions are, see

Re: [Wikitech-l] Hard deprecation of the Revision class

2020-07-11 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA256 Hi, On 2020-07-06 13:18, Petr Pchelko wrote: > TLDR: your extension CI might break due to hard deprecation of > Revision class. Unfortunately, this is happening to a significant amount of extensions. The latest libup security run shows failures acr

[Wikitech-l] Help test the new MediaWiki codesearch UI

2020-07-31 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA256 Hi, tl;dr: Help would be appreciated testing a new MediaWiki codesearch UI: . Sticking "-beta" in existing URLs should just work. The current codesearch interface is a pretty bad hack based on upstream's UI. Or

Re: [Wikitech-l] [MediaWiki-l] MediaWiki 1.35.0-rc.0 is ready for testing

2020-08-02 Thread Kunal Mehta
-BEGIN PGP SIGNED MESSAGE- Hash: SHA256 Hi, On 2020-07-31 07:05, Sam Reed wrote: > I'm pleased to announce the immediate availability of MediaWiki > 1.35.0-rc.0, the first release candidate for 1.35.x, the next LTS > version to replace 1.31 which is due to go end of life in June > 2021. D

Re: [Wikitech-l] Making breaking changes without deprecation?

2020-08-31 Thread Kunal Mehta
Hi, On 2020-08-28 02:18, Daniel Kinzler wrote: > tl;dr: the key question is: > > Can we shorten or even entirely skip the deprecation process, > if we have removed all usages of the obsolete code from public > extensions? I think going down this road would be a mistake, mostly becaus

Re: [Wikitech-l] Technical Decision Making Process Update and Change

2021-01-28 Thread Kunal Mehta
Hi, On 1/25/21 6:19 PM, Kate Chapman wrote: If you currently have an RFC in process with TechCom that is not on Last Call, it may need to be moved into this process. If you have filed an RFC that is no longer relevant please close it. The group setting up the process will be inquiring about the

[Wikitech-l] Performance regression expected with Debian Buster upgrade

2021-02-03 Thread Kunal Mehta
Hi, We're currently in the process of upgrading the MediaWiki servers to Debian Buster and expect a performance regression to come with it. The cause appears to be better Spectre[1] mitigations in the Buster 4.19 kernel, which we can't disable. Most of the effect is seen in code that ends up

Re: [Wikitech-l] GlobalWatchlist enabled on testwiki

2021-02-09 Thread Kunal Mehta
Hi, On 2/9/21 4:44 PM, Danny S wrote: I developed the extension to replace a user script I wrote[5], funded through a grant from the WMF[6], and the deployment to meta follows 8 months of work on the extension, and an additional year of work on the script.[7] This is awesome, thank you so m

  1   2   >