profile
viewpoint
Kunal Mehta legoktm https://legoktm.com Most projects are available from https://git.legoktm.com/

dtzWill/xkcd-hash 4

Code for xkcd hash competition

legoktm/adminbots 3

A collection of IRC bots to assist with administrative tasks on Wikipedia

legoktm/checker 3

checker!

legoktm/fab 3

Phabricator API wrapper

legoktm/dotfiles 1

My dotfiles

legoktm/acousticbrainz-client 0

A client to upload data to an acousticbrainz server

legoktm/ADB 0

AutoDataBrowser

legoktm/afch-updater 0

Script to automatically update afch on test.wikipedia.org

legoktm/AutoCAS 0

Autmatically fill in values in the patient log for Case med students

push eventwikimedia/mediawiki

Arlo Breault

commit sha b97b86e0eae8935b3c90e29aa27175fdaf44897f

Revert most of the changes from 16b76a4 At the time, Parsoid only emitted the figcaption if a caption was present. So, having the border set on the figcaption was leading to some incomplete borders on figures. However, since I856ee962b70cef1f8d49652396ea5264e11a8ade, that's no longer the case. We can now switch back to displaying the figcaption as a table-caption and don't have to rely on the hacky `width: 1px` to have the figcaption width match the media in the figure (see https://stackoverflow.com/a/6536025). One problem with the 1px hack is that in T241392, TimedMediaHandler sets the max-width of a video to 100%, which shrinks it down to that 1px. The change in display for the figcaption necessitates a bit of trickery to get the magnify clip in the right place. At the time of the reverted commit, css for the magnify clip wasn't present. Bug: T272186 Bug: T266149 Change-Id: If283c1ab200ca1049343cdac759d7a2b78306227

view details

jenkins-bot

commit sha 86092cf0735dbb6ced5047305cfedbf4dd33699c

Merge "Revert most of the changes from 16b76a4"

view details

push time in an hour

pull request commenthound-search/hound

Install dependencies in Dockerfile

Hey there! I'm sorry you're running in to issues here. We recently moved dependencies to go modules, and semver is now listed as a dependency in our go.mod file. I'm relatively new to both go modules and Docker, though, so there may be an extra step we need to add to the dockerfile to ensure dependencies are installed. That being said, a go get step might not be the right approach here, since we'd need to add a go get for every other dependency.

I'll leave this PR open for now, but I may turn it into an issue if we can't figure an answer out shortly. Thank you for bringing this up!

yayachiken

comment created time in an hour

push eventwikimedia/pywikibot

Bharatkhatri

commit sha 2dd91c5f0537888ef66d6c644df4533a16907449

Removed remove magic coding comments # -*- coding: utf-8 -*- Bug: T270946 Change-Id: Ie680a3ef658aa38e551a804b3aa4f85356d3adc0

view details

jenkins-bot

commit sha 73aa969abb74a6b54461302b4bca7dcc1a60b38d

Merge "Removed remove magic coding comments # -*- coding: utf-8 -*-"

view details

push time in 2 hours

PR opened hound-search/hound

Install dependencies in Dockerfile

This fixes the Dockerfile such that building the container image is possible using something like:

docker build .

Without adding the semver package, that build command would fail with the message

go/src/github.com/hound-search/hound/cmds/houndd/main.go:17:2: cannot find package "github.com/blang/semver" in any of:
	/usr/lib/go/src/github.com/blang/semver (from $GOROOT)
	/go/src/github.com/blang/semver (from $GOPATH)
The command '/bin/sh -c apk update 	&& apk add go git subversion libc-dev mercurial bzr openssh 	&& go install github.com/hound-search/hound/cmds/houndd 	&& apk del go 	&& rm -f /var/cache/apk/* && rm -rf /go/src /go/pkg' returned a non-zero code: 1

Note that I am not super-knowledgeable about the golang ecosystem. If it is possible to build the image without such changes, please tell me, I could then add the information to the README.

In addition, the pinned alpine image was bumped to the latest bugfix release, but the build works without that specific change.

+2 -1

0 comment

1 changed file

pr created time in 2 hours

startedcharmbracelet/glow

started time in 3 hours

pull request commenthound-search/hound

Added new folder docs with config-options description on options.

Actually, it looks like I can apply changes myself. Let me know if you're ok with them and I'll merge this branch in.

ehrktia

comment created time in 3 hours

startedmuesli/termenv

started time in 3 hours

Pull request review commenthound-search/hound

Added new folder docs with config-options description on options.

+- [ConfigOptions](#configoptions)+  * [Git options](#git-options)+  * [SVN options](#svn-options)+  * [URL options](#url-options)++++### ConfigOptions+Various options available in `config.json` are used by hound.Comprehensive list of available options are provided below with some sane defaults.+keys used in the config json file are the options,description provides details about keys.Default values gives idea about value which can be used for the option.+++ConfigOption | Description | Default Values+:------ | :----- | :-----+max-concurrent-indexers | defines the total number of indexers required to be used for indexing code | 2+health-check-uri |  health check url for hound | `/healthz`+dbpath | absolute file path where the `config.json` file exists| `data`+title | Title used for the application | Hound+url-pattern | composed of base url and anchor values in form of key value pairs | n/a+vcs-config | holds the version control config, default VCS used in Hound is git.Other options for VCS are svn,mercurial,bitbucket,hg, etc.Refer to `config-example.json` to get the list of vcs and usage. Below tables provide detailed options list of each type of vcs | git+repos | holds the list of repos which are required to be indexed by Hound . Each Repo is added with reponame as a Json Key with options associated with repo as values similar to example provided in `config-example.json` | n/a++#### Git options
## Git Options
ehrktia

comment created time in 3 hours

Pull request review commenthound-search/hound

Added new folder docs with config-options description on options.

+- [ConfigOptions](#configoptions)+  * [Git options](#git-options)+  * [SVN options](#svn-options)+  * [URL options](#url-options)++++### ConfigOptions+Various options available in `config.json` are used by hound.Comprehensive list of available options are provided below with some sane defaults.+keys used in the config json file are the options,description provides details about keys.Default values gives idea about value which can be used for the option.+++ConfigOption | Description | Default Values+:------ | :----- | :-----+max-concurrent-indexers | defines the total number of indexers required to be used for indexing code | 2+health-check-uri |  health check url for hound | `/healthz`+dbpath | absolute file path where the `config.json` file exists| `data`+title | Title used for the application | Hound+url-pattern | composed of base url and anchor values in form of key value pairs | n/a+vcs-config | holds the version control config, default VCS used in Hound is git.Other options for VCS are svn,mercurial,bitbucket,hg, etc.Refer to `config-example.json` to get the list of vcs and usage. Below tables provide detailed options list of each type of vcs | git+repos | holds the list of repos which are required to be indexed by Hound . Each Repo is added with reponame as a Json Key with options associated with repo as values similar to example provided in `config-example.json` | n/a++#### Git options+List of options associated with git vcs in repos++GitOptions  | Description | Default Values+:------ | :----- | :-----+ms-between-polls | time interval to poll the repo url | 30s+detect-ref    | used to determine branch |  master branch +ref | used to provide reference for the branch for repo| n/a++#### SVN options++List of options available for SVN vcs in repos++SvnOptions  | Descriptions| Default Values+:------ | :-----| :-----+username  | user name for the svn repo | n/a+password | password to authenticate use for svn repo | n/a+++#### URL options 
## URL Options 
ehrktia

comment created time in 3 hours

Pull request review commenthound-search/hound

Added new folder docs with config-options description on options.

+- [ConfigOptions](#configoptions)+  * [Git options](#git-options)+  * [SVN options](#svn-options)+  * [URL options](#url-options)++++### ConfigOptions+Various options available in `config.json` are used by hound.Comprehensive list of available options are provided below with some sane defaults.+keys used in the config json file are the options,description provides details about keys.Default values gives idea about value which can be used for the option.+++ConfigOption | Description | Default Values+:------ | :----- | :-----+max-concurrent-indexers | defines the total number of indexers required to be used for indexing code | 2+health-check-uri |  health check url for hound | `/healthz`+dbpath | absolute file path where the `config.json` file exists| `data`+title | Title used for the application | Hound+url-pattern | composed of base url and anchor values in form of key value pairs | n/a+vcs-config | holds the version control config, default VCS used in Hound is git.Other options for VCS are svn,mercurial,bitbucket,hg, etc.Refer to `config-example.json` to get the list of vcs and usage. Below tables provide detailed options list of each type of vcs | git+repos | holds the list of repos which are required to be indexed by Hound . Each Repo is added with reponame as a Json Key with options associated with repo as values similar to example provided in `config-example.json` | n/a++#### Git options+List of options associated with git vcs in repos++GitOptions  | Description | Default Values+:------ | :----- | :-----+ms-between-polls | time interval to poll the repo url | 30s+detect-ref    | used to determine branch |  master branch +ref | used to provide reference for the branch for repo| n/a++#### SVN options
## SVN Options
ehrktia

comment created time in 3 hours

Pull request review commenthound-search/hound

Added new folder docs with config-options description on options.

+- [ConfigOptions](#configoptions)+  * [Git options](#git-options)+  * [SVN options](#svn-options)+  * [URL options](#url-options)++++### ConfigOptions

Nit:

# Config Options
ehrktia

comment created time in 3 hours

Pull request review commenthound-search/hound

Added new folder docs with config-options description on options.

+- [ConfigOptions](#configoptions)+  * [Git options](#git-options)+  * [SVN options](#svn-options)+  * [URL options](#url-options)++++### ConfigOptions+Various options available in `config.json` are used by hound.Comprehensive list of available options are provided below with some sane defaults.

Nit:

Most of Hound's behavior is defined by a single `config.json` configuration file. A list of its available options are provided below, along with their defaults.
ehrktia

comment created time in 3 hours

Pull request review commenthound-search/hound

Added new folder docs with config-options description on options.

 the config found in [default-config.json](default-config.json): } ``` ++complete list of available config options can be found under [config-options](docs/config-options.md).  

Nit:

A complete list of available config options can be found [here](docs/config-options.md).  
ehrktia

comment created time in 3 hours

push eventwikimedia/mediawiki

Ammarpad

commit sha beeccf852082b94505da98d03dff7c8468af76c4

LinkBatch: Fix property documentations * Remove incorrect null type from documentation of properties that cannot be null. * Add type doc for $caller and $data Change-Id: I8d51dca7fd4910acf744361279c441fe4c3a4307

view details

jenkins-bot

commit sha 7684e0f2d3fdd9aefdcffbcb0618a8543e12eb74

Merge "LinkBatch: Fix property documentations"

view details

push time in 4 hours

push eventwikimedia/pywikibot

JJMC89

commit sha c46dafed8f1de54cd829db3ac679fea31621ec24

[tests] parentid not in all alldeletedrevisions responses Bug: T272711 Change-Id: Ibd6f461223e242967425ac8699896f8ce7676bc9

view details

push time in 4 hours

push eventwikimedia/mediawiki

Thalia

commit sha d8356d741c3502bb73f39d95fec737fffad990e1

HTMLFileField: Make 'accept' parameter optional Change-Id: I55321f90fad832d22661628334d2d254780e07da

view details

push time in 4 hours

push eventwikimedia/mediawiki

Ed Sanders

commit sha 744569f8b6c9ef9e3dcdfa40f4953845c944b8ab

Follow-up: I21d17de56: HtmlFileField fixes * Make 'accept' an array like it is in OOUI * Treat $this->mMultiple like the boolean it is * Don't bother setting 'placeholder' on native inputs Change-Id: I4c1341181757791c2e1ac2a14c4b3e7c8461ca54

view details

push time in 5 hours

push eventwikimedia/mediawiki

Ed Sanders

commit sha ee4f008be950bf50d1c59a7bce80a13141387581

Move select file messages from oojs-ui-widgets to oojs-ui-core These messages are now used by SelectFileInputWidget which is part of core, not widgets (as it has a PHP version). Change-Id: I062e12b8dd80b34b3aad54dd65e5026de3c69dbe

view details

push time in 5 hours

push eventwikimedia/pywikibot

xqt

commit sha 408f3277558f4fe4559a88177a60379685813fc6

[tests] Fix assertRaises tests in api_tests.py Bug: T267801 Change-Id: I70e650df3ffe488bf8ccfb029c0844bf61f7e4ae

view details

Xqt

commit sha 1eaa9d4b57c5c2e2f800f72919f11c7a896417c3

Merge "[tests] Fix assertRaises tests in api_tests.py"

view details

push time in 5 hours

push eventwikimedia/pywikibot

xqt

commit sha ba475f3d613f6f55bd043dcfbf5f639fb4ed17d4

[bugfix] use unittest.TestCase.skipTest Fix spelling mistake Change-Id: If2dd19013610a2521703bac90b5dfbb9ff36b409

view details

jenkins-bot

commit sha 63af650e13f329e81c7000d81d2e657f18ccb705

Merge "[bugfix] use unittest.TestCase.skipTest"

view details

push time in 7 hours

push eventwikimedia/pywikibot

homeboy445

commit sha 07578c679ce3fad767a7246de6559bd321e6e252

Adopted context Manager call for assertRaises in certain script(s) Bug: T267801 Change-Id: I091828126064f3e4a47f0ad6671d202a9a969da3

view details

jenkins-bot

commit sha d0ec8edea65ed3c8cf05c0166492e545a4596657

Merge "Adopted context Manager call for assertRaises in certain script(s)"

view details

push time in 7 hours

push eventwikimedia/pywikibot

Amir Sarabadani

commit sha 8f6830eb31e3a0f46c71c12d89218360666637da

Remove als from closed wikibooks This wiki does not exist, moved them to removed Bug: T272668 Change-Id: Ia86ab0d70eebdc4cc9f3fe66d2ae96c24c7e3776

view details

jenkins-bot

commit sha 926411dc61d07a1d30cf1ba555786f1b02535fe0

Merge "Remove als from closed wikibooks"

view details

push time in 8 hours

push eventwikimedia/mediawiki

Umherirrender

commit sha b9b844f352db08e3467875efffb5f9b548b99f07

Special:Contributions reports negative namespace error on php8 Comparing the empty string with a number fails on php8 https://wiki.php.net/rfc/saner-numeric-strings Bug: T248925 Change-Id: I9581833ed31e0846933041c1698953c7eb82c20e

view details

jenkins-bot

commit sha 2183d3baa2ef63b9c64dfd2c5a823815cb66ed46

Merge "Special:Contributions reports negative namespace error on php8"

view details

push time in 8 hours

push eventwikimedia/pywikibot

Amir Sarabadani

commit sha dc23cfad197e3af10a2482dbd0a180685b7a6d6b

Remove als from closed wikis of wikiquote and wiktionary These wikis are not closed, they don't exist, moved them to removed Bug: T272668 Change-Id: I960e6895843dfcd7dc70ae5176c2c8172532ed59

view details

push time in 8 hours

push eventwikimedia/mediawiki

Ariel T. Glenn

commit sha a411a8da9ab2bb07b57aa21647758bd0c08ec98a

set revision Title before setting text, or there may be no content model with the reverse order, the content model of the rvision is null, and it can't be gotten from the title, so dumpIterator dies when the file option is used instead of a dump. Change-Id: Ia67097565953f00068399da3225b939fb105ba4a

view details

jenkins-bot

commit sha a1309f6d4051a5dbf586fdacdaee8db92721aaf6

Merge "set revision Title before setting text, or there may be no content model"

view details

push time in 8 hours

push eventwikimedia/mediawiki

Ariel T. Glenn

commit sha 6df4b78e36af1c94dfebd28a27a4b53d1b0473f1

WikiRevision::model can be null so don't pass it as an arg Use getModel() as the arg instead Change-Id: Ie23d31885416abdcc639f4fc5028961ba80de495

view details

jenkins-bot

commit sha b996a9d3d53b639268da6d521b59e45fbc900720

Merge "WikiRevision::model can be null so don't pass it as an arg"

view details

push time in 8 hours

push eventwikimedia/mediawiki

DannyS712

commit sha 2086bfa36ed1dd97c0a2e6a9a42825cda100d898

DefaultSettings: Update $wgUseTagFilter documentation Remove `valid_tag` reference, table no longer exists Change-Id: Ie2d4d592bf42b44266c9e8e99186f31341ff46bd

view details

jenkins-bot

commit sha 0fa469f18527cabba86a7569996aad14bf4b74fd

Merge "DefaultSettings: Update $wgUseTagFilter documentation"

view details

push time in 9 hours

push eventwikimedia/integration-config

DannyS712

commit sha c30e2c2391c658449c38839288bb33b445c1da23

Add `mark` to mediawiki-i18n-check-docker html whitelist Bug: T272671 Change-Id: Ifeea03e908cfc4b3cfd92c4da4e989986b789efe

view details

push time in 10 hours

push eventwikimedia/mediawiki

Petr Pchelko

commit sha 10de1d347280f54738e923f2ac02611557da7a0c

Introduce MockAuthorityTrait for Authority test helpers Change-Id: I152f2cbe8a7000eedc237fb46b4e4c6d2f9730d4

view details

jenkins-bot

commit sha bc830dc0d0dd4657aa49ca82ef98dbafd9117266

Merge "Introduce MockAuthorityTrait for Authority test helpers"

view details

push time in 11 hours

push eventwikimedia/mediawiki

Petr Pchelko

commit sha 3a2e8883b42ccfeaa215bb83a97dc0608aae6d93

Rest: use Authority in all core handlers Bug: T239753 Change-Id: Idf2229255f49514dd8b68bf63573c5b619b4f2f1

view details

jenkins-bot

commit sha 9dbdf712a49864645fe81adf4a0327e4af0b28a1

Merge "Rest: use Authority in all core handlers"

view details

push time in 11 hours

more