vm (Vihang Mehta)
User

Projects

User does not belong to any projects.

Calendar

User Details

User Since: Jan 18 2013, 4:25 AM (626 w, 1 d)
Availability: Available

Recent Activity
View All

Jun 7 2022

vm abandoned D10443: Allow arc patch to keep empty commits.

Jun 7 2022, 9:42 PM

vm abandoned D15124: Add linter to support eslint.

Jun 7 2022, 9:42 PM

vm requested review of D21853: Fix more PHP 8.1 "strlen(null)" callsites.

Jun 7 2022, 7:48 PM

vm added a revision to T13588: PHP 8 Compatibility: D21853: Fix more PHP 8.1 "strlen(null)" callsites.

Jun 7 2022, 7:48 PM · Infrastructure

Feb 10 2021

vm updated the summary of D21552: Update golint install instructions.

Feb 10 2021, 6:01 PM

vm requested review of D21552: Update golint install instructions.

Feb 10 2021, 6:00 PM

Jan 28 2016

vm added a comment to D15124: Add linter to support eslint.

In D15124#173386, @avivey wrote:

See D12198, T9785, and T10038.

Jan 28 2016, 1:09 AM

vm retitled D15124: Add linter to support eslint from to Add linter to support eslint.

Jan 28 2016, 12:16 AM

vm retitled D15123: Fix linter tests from to Fix linter tests.

Jan 28 2016, 12:14 AM

Oct 2 2015

vm retitled D14225: Set name on more linters from Set path on more linters to Set name on more linters.

Oct 2 2015, 4:10 PM

vm retitled D14225: Set name on more linters from to Set path on more linters.

Oct 2 2015, 3:57 PM

vm retitled D14220: Fix ArcanistClosureLinter from to Fix ArcanistClosureLinter.

Oct 2 2015, 3:30 AM

vm added a revision to T9498: ArcanistClosureLinter broken after D14165: D14220: Fix ArcanistClosureLinter.

Oct 2 2015, 3:30 AM · Lint

vm created T9498: ArcanistClosureLinter broken after D14165.

Oct 2 2015, 3:23 AM · Lint

Feb 11 2015

vm edited the content of Organizations Using Phabricator.

Feb 11 2015, 8:01 AM

Nov 5 2014

vm added a revision to T5986: arc patch with dependent diffs is strange: D10443: Allow arc patch to keep empty commits.

Nov 5 2014, 7:33 PM · Arcanist

vm added a task to D10443: Allow arc patch to keep empty commits: T5986: arc patch with dependent diffs is strange.

Nov 5 2014, 7:33 PM

Oct 15 2014

vm added a comment to D10443: Allow arc patch to keep empty commits.

any updates?

Oct 15 2014, 6:56 AM

Sep 25 2014

vm added a comment to D10443: Allow arc patch to keep empty commits.

ping?

Sep 25 2014, 4:26 PM

Sep 8 2014

vm retitled D10443: Allow arc patch to keep empty commits from to Allow arc patch to keep empty commits.

Sep 8 2014, 6:22 PM

Sep 2 2014

vm added a comment to T5986: arc patch with dependent diffs is strange.

Another problem I notices recently, say you have D2 depending on D1, and D1 gets reviewed and landed into HEAD.
arc patch D2 still tries to cherry-pick D1 and then fails since you are now trying to cherry pick an empty commit.

Sep 2 2014, 6:01 PM · Arcanist

Sep 1 2014

vm updated the diff for D10381: Pass conduit credentials down to children workflow.

Remove debugging code

Sep 1 2014, 2:48 AM

vm added inline comments to D10381: Pass conduit credentials down to children workflow.

Sep 1 2014, 2:47 AM

Aug 28 2014

vm added a revision to T5986: arc patch with dependent diffs is strange: D10381: Pass conduit credentials down to children workflow.

Aug 28 2014, 7:17 PM · Arcanist

vm added a task to D10381: Pass conduit credentials down to children workflow: T5986: arc patch with dependent diffs is strange.

Aug 28 2014, 7:17 PM

vm retitled D10381: Pass conduit credentials down to children workflow from to Pass conduit credentials down to children workflow.

Aug 28 2014, 7:16 PM

vm created T5986: arc patch with dependent diffs is strange.

Aug 28 2014, 9:21 AM · Arcanist

vm created T5984: Blocking Others list is not accurate with Project Reviewers enabled.

Aug 28 2014, 7:06 AM · Phabricator

Aug 26 2014

avivey awarded D10352: Allow lines to be specified as comma delimited number ranges for arc browse a Like token.

Aug 26 2014, 4:34 PM

vm added a revision to T5971: Allow multiline selection via `arc browse`: D10352: Allow lines to be specified as comma delimited number ranges for arc browse.

Aug 26 2014, 3:17 PM · Arcanist

vm retitled D10352: Allow lines to be specified as comma delimited number ranges for arc browse from to Allow lines to be specified as comma delimited number ranges for arc browse.

Aug 26 2014, 3:17 PM

Aug 25 2014

vm awarded D10346: Fix `arc browse path:line` a Grey Medal token.

Aug 25 2014, 7:54 PM

Aug 18 2014

vm added a comment to T5781: Moar functionality for `arc browse`!.

D10140 broke arc browse filename:<line-numbers>
It returns Unable to find an object named heading.js:66, no such commit exists in the remote, and no such path exists in the working copy. Use --force to treat this as a path anyway.

Aug 18 2014, 11:49 PM · Arcanist

Aug 8 2014

vm awarded D10198: Remove --background flag to prevent `arc` from hanging a Manufacturing Defect? token.

Aug 8 2014, 9:57 PM

May 23 2014

vm added a comment to T4281: Restore background linting and unit tests for performance.

Note: This reproed even with no lint errors.

May 23 2014, 5:32 PM · Arcanist

vm added a comment to T4281: Restore background linting and unit tests for performance.

A couple of coworkers have been complaining about arc diff hanging at 'Linting ...', I have a feeling it might be the same bug as this one since --background 0 allows the diffs to go through.
I will try to get a helpful stack trace, but meanwhile I just wanted to ping this thread as a heads up.

May 23 2014, 5:32 PM · Arcanist

May 15 2014

vm updated the diff for D9131: Add Closure linter to arcanist.

Add test case
Also small changes as per comments

May 15 2014, 4:17 PM

vm retitled D9131: Add Closure linter to arcanist from to Add Closure linter to arcanist.

May 15 2014, 10:01 AM

Feb 20 2014

vm added a comment to Image Macro "rickroll".

Feb 20 2014, 9:42 PM

vm attached audio to Image Macro "rickroll": Unknown Object (File).

Feb 20 2014, 9:41 PM

vm changed the image for Image Macro "rickroll".

Feb 20 2014, 9:41 PM