Page MenuHomePhabricator

shrinidhirao (Shrinidhi Rao)
User

Projects

User does not belong to any projects.

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Saturday

  • Clear sailing ahead.

User Details

User Since
Aug 20 2014, 4:44 PM (505 w, 1 d)
Availability
Available

Recent Activity

Dec 17 2019

shrinidhirao edited the content of Organizations Using Phabricator.
Dec 17 2019, 7:21 PM

Feb 12 2016

shrinidhirao created T10345: Previous and Next post links in blog.phacility.com encounter exceptions.
Feb 12 2016, 6:18 AM · Phame, Phacility, Bug Report

Aug 3 2015

shrinidhirao added a comment to T8672: PhabricatorTaskmasterDaemon fatals in PublishWorker.

Is this right?

Aug 3 2015, 10:53 AM · Diffusion, Daemons

Jul 7 2015

shrinidhirao awarded D13444: Provide phutil_json_encode() a Like token.
Jul 7 2015, 2:51 PM

Jun 25 2015

shrinidhirao created T8672: PhabricatorTaskmasterDaemon fatals in PublishWorker.
Jun 25 2015, 9:58 AM · Diffusion, Daemons

Jun 12 2015

shrinidhirao edited the content of Organizations Using Phabricator.
Jun 12 2015, 7:06 PM

Jun 2 2015

shrinidhirao added a comment to T7447: Bring inline comments forward across revision updates.

This is really helping to improve the code readability. A big welcome change :)
A small learning curve to first discover the "hide comments" feature was needed in the team. [Maybe "X" was making people hesitant].
Not many noises to have the Hide All/Show All feature, but as you said it is a personal opinion.

Jun 2 2015, 4:04 PM · Prioritized, Differential

May 29 2015

shrinidhirao awarded T8302: Maybe allow ctrl-click to search for any symbol? a Like token.
May 29 2015, 9:36 AM · Symbols

May 15 2015

shrinidhirao added a comment to T7447: Bring inline comments forward across revision updates.

I just want to give a big +1 to this. Even though we don't practice two reviewers, I find it very useful to not be distracted with inline comments when doing a final, read-the-code-undistracted review. Just being able to globally disable forward-transferring comments isn't very useful to us: It totally disables a generally useful feature. Being able to temporarily disable inline comments would help a lot, i.e. as I mentioned before by adding a "view options" for the diff (preferably in addition to a keyboard shortcut, but just a keyboard shortcut is too obscure IMHO).

I second the suggestion to hide all inline comments in a diff view. This does give a easy reading if there are many comments one-after-one.
Something on the lines of D12650.

May 15 2015, 3:06 PM · Prioritized, Differential

May 4 2015

shrinidhirao added a comment to T7447: Bring inline comments forward across revision updates.

Combining a coversation to a single box would also help.
+1 to links to the original context.

May 4 2015, 6:04 PM · Prioritized, Differential

Apr 30 2015

shrinidhirao closed T8003: Total vote count is one less than sum of all options - V27 as Invalid.

Oops... didn't know this.

Apr 30 2015, 5:21 PM · Slowvote
shrinidhirao updated the task description for T8003: Total vote count is one less than sum of all options - V27.
Apr 30 2015, 5:12 PM · Slowvote
shrinidhirao created T8003: Total vote count is one less than sum of all options - V27.
Apr 30 2015, 5:10 PM · Slowvote

Apr 25 2015

shrinidhirao added a comment to T7917: Better Duplicate Icon?.

#1 has a degree of intuition.

Apr 25 2015, 8:06 AM · Maniphest

Apr 1 2015

shrinidhirao added a comment to T4279: Improve auth/LDAP import tools to assist in linking/merging accounts and migrations across providers.

Log in first using the OLD method.
Then under user settings settings/panel/external/ link LDAP account.

Apr 1 2015, 1:06 PM · Auth

Mar 25 2015

shrinidhirao created T7658: Deleting a draft inline comment and adding new inline comment moves the new comment to end of file..
Mar 25 2015, 5:06 AM · Differential

Jan 22 2015

shrinidhirao closed T7009: hello this is demo as Spite.
Jan 22 2015, 6:26 PM

Jan 18 2015

shrinidhirao added a comment to T6743: Loosen arcanist requirements for matching SVN diffs.

1seth@luca:~/update/build$ arc which --trace
2libphutil loaded from '/home/seth/local/src/arcanist/libphutil/src'.
3arcanist loaded from '/home/seth/local/src/arcanist/arcanist/src'.
4Config: Reading user configuration file "/home/seth/.arcrc"...
5Config: Did not find system configuration at "/etc/arcconfig".
6Working Copy: Reading .arcconfig from "/home/seth/update/build/.arcconfig".
7Working Copy: Path "/home/seth/update/build" is part of `svn` working copy "/home/seth/update/build".
8Working Copy: Project root is at "/home/seth/update/build".
9Config: Did not find local configuration at "/home/seth/update/build/.svn/arc/config".
10>>> [0] <conduit> conduit.connect() <bytes = 467>
11>>> [1] <http> http://phab.spira:8001/api/conduit.connect
12<<< [1] <http> 63,213 us
13<<< [0] <conduit> 63,617 us
14REPOSITORY
15>>> [2] <conduit> repository.query() <bytes = 190>
16>>> [3] <http> http://phab.spira:8001/api/repository.query
17<<< [3] <http> 36,183 us
18<<< [2] <conduit> 36,339 us
19To identify the repository associated with this working copy, arc followed this process:
20
21 Configuration value "repository.callsign" is set to "M".
22
23 Found a unique matching repository.
24
25This working copy is associated with the M repository.
26
27MATCHING REVISIONS
28These Differential revisions match the changes in this working copy:
29
30 (No revisions match.)
31
32Since there are no revisions in Differential which match this working copy, a
33new revision will be created if you run 'arc diff'.
34
35seth@luca:~/update/build$ echo 'hello' >> makefile.common
36seth@luca:~/update/build$ arc diff
37Linting...
38No lint engine configured for this project.
39Running unit tests...
40No unit test engine is configured for this project.
41Created a new Differential revision:
42 Revision URI: http://phab.spira:8001/D7
43
44Included changes:
45 M makefile.common
46seth@luca:~/update/build$ arc which --trace
47libphutil loaded from '/home/seth/local/src/arcanist/libphutil/src'.
48arcanist loaded from '/home/seth/local/src/arcanist/arcanist/src'.
49Config: Reading user configuration file "/home/seth/.arcrc"...
50Config: Did not find system configuration at "/etc/arcconfig".
51Working Copy: Reading .arcconfig from "/home/seth/update/build/.arcconfig".
52Working Copy: Path "/home/seth/update/build" is part of `svn` working copy "/home/seth/update/build".
53Working Copy: Project root is at "/home/seth/update/build".
54Config: Did not find local configuration at "/home/seth/update/build/.svn/arc/config".
55>>> [0] <conduit> conduit.connect() <bytes = 467>
56>>> [1] <http> http://phab.spira:8001/api/conduit.connect
57<<< [1] <http> 88,854 us
58<<< [0] <conduit> 89,045 us
59REPOSITORY
60>>> [2] <conduit> repository.query() <bytes = 190>
61>>> [3] <http> http://phab.spira:8001/api/repository.query
62<<< [3] <http> 64,252 us
63<<< [2] <conduit> 64,404 us
64To identify the repository associated with this working copy, arc followed this process:
65
66 Configuration value "repository.callsign" is set to "M".
67
68 Found a unique matching repository.
69
70This working copy is associated with the M repository.
71
72MATCHING REVISIONS
73These Differential revisions match the changes in this working copy:
74
75 (No revisions match.)
76
77Since there are no revisions in Differential which match this working copy, a
78new revision will be created if you run 'arc diff'.
79seth@luca:~/update/build$ svn info .
80Path: .
81URL: svn://svn.spira/mainrepo/build/trunk
82Repository Root: svn://svn.spira/mainrepo
83Repository UUID: 5b050b1a-ec2b-4496-a8ad-8719d879b7e6
84Revision: 27456
85Node Kind: directory
86Schedule: normal
87Last Changed Author: seth
88Last Changed Rev: 152
89Last Changed Date: 2014-12-12 15:39:17 -0500 (Fri, 12 Dec 2014)

Jan 18 2015, 9:09 AM · Arcanist, Subversion

Dec 19 2014

shrinidhirao added a comment to T6782: Always context not available.

You can use gnu-diff and a workaround to get the full context in svn diff.
Something on the lines of [T2465#24935]

Dec 19 2014, 11:21 AM · Differential

Dec 18 2014

shrinidhirao added a comment to T6782: Always context not available.

What VCS are you using - svn , git or mercurial?

Dec 18 2014, 3:04 PM · Differential

Nov 27 2014

shrinidhirao added a project to T6658: Unhandled Exception ("PhabricatorApplicationTransactionNoEffectException") while creating diff via web interface: Transactions.
Nov 27 2014, 4:20 PM · Transactions, Differential
shrinidhirao created T6658: Unhandled Exception ("PhabricatorApplicationTransactionNoEffectException") while creating diff via web interface.
Nov 27 2014, 4:19 PM · Transactions, Differential

Oct 17 2014

shrinidhirao added a comment to T5029: Fill in missing context for low-context diffs against tracked repositories.

Was about to create a new task for same thing for SVN working copies.

Oct 17 2014, 9:06 AM · KDE, Restricted Project, Diffusion, Differential

Oct 8 2014

shrinidhirao added a comment to T6261: Remove Gravatar support to reduce setup complexity on hosts without access to the internet.

Perfect!!! thank you!

Oct 8 2014, 10:31 AM · People, Support Impact

Aug 20 2014

shrinidhirao added a comment to T5925: Comment export option in differential.

Thanks for the information.

Aug 20 2014, 5:19 PM · Facts
shrinidhirao created T5925: Comment export option in differential.
Aug 20 2014, 4:45 PM · Facts