navaneeth.suresh (Navaneeth Suresh)
User

Projects

User does not belong to any projects.

User Details

User Since
Oct 10 2018, 6:12 AM (23 w, 5 d)

Recent Activity

Yesterday

navaneeth.suresh created D6169: unshelve: disable unshelve during merge (issue5123).
Mon, Mar 25, 3:17 AM

Sun, Mar 24

navaneeth.suresh added a comment to D6027: mq: make unshelve to apply on modified mq patch (issue4318).

Gentle ping for review.

Sun, Mar 24, 2:09 PM
navaneeth.suresh added a comment to D6056: patch: stop aborting when add/rename/copy files on --interactive (issue5727).

Gentle ping for review.

Sun, Mar 24, 2:08 PM

Sun, Mar 17

navaneeth.suresh added inline comments to D6056: patch: stop aborting when add/rename/copy files on --interactive (issue5727).
Sun, Mar 17, 12:59 AM

Sat, Mar 16

navaneeth.suresh added inline comments to D6056: patch: stop aborting when add/rename/copy files on --interactive (issue5727).
Sat, Mar 16, 4:57 AM

Thu, Mar 14

navaneeth.suresh updated the diff for D6056: patch: stop aborting when add/rename/copy files on --interactive (issue5727).
Thu, Mar 14, 2:21 AM

Fri, Mar 8

navaneeth.suresh updated the diff for D6027: mq: make unshelve to apply on modified mq patch (issue4318).
Fri, Mar 8, 1:26 PM

Thu, Mar 7

navaneeth.suresh added inline comments to D6069: uncommit: add flag --allow-dirty-working-copy.
Thu, Mar 7, 2:51 PM

Wed, Mar 6

navaneeth.suresh closed D6069: uncommit: add flag --allow-dirty-working-copy.
Wed, Mar 6, 8:39 AM
navaneeth.suresh closed D5940: uncommit: don't allow dirty working copy with PATH (issue5977).
Wed, Mar 6, 8:39 AM
navaneeth.suresh committed rHGaa284d9a33ca: uncommit: add flag --allow-dirty-working-copy.
uncommit: add flag --allow-dirty-working-copy
Wed, Mar 6, 8:39 AM
navaneeth.suresh committed rHGbf22e370ae9a: uncommit: don't allow dirty working copy with PATH (issue5977).
uncommit: don't allow dirty working copy with PATH (issue5977)
Wed, Mar 6, 8:39 AM
navaneeth.suresh added a dependent revision for D5940: uncommit: don't allow dirty working copy with PATH (issue5977): D6069: uncommit: add flag --allow-dirty-working-copy.
Wed, Mar 6, 7:32 AM
navaneeth.suresh added a dependency for D6069: uncommit: add flag --allow-dirty-working-copy: D5940: uncommit: don't allow dirty working copy with PATH (issue5977).
Wed, Mar 6, 7:32 AM
navaneeth.suresh added a comment to D5940: uncommit: don't allow dirty working copy with PATH (issue5977).

Can you fix the first line of the commit message to say "don't allow dirty working copy with PATH" (the "don't" was missing). It was allowed before this patch, it is not allowed with this patch (unless the config is set, of course).

Wed, Mar 6, 12:50 AM
navaneeth.suresh retitled D5940: uncommit: don't allow dirty working copy with PATH (issue5977) from uncommit: allow dirty working copy with PATH (issue5977) to uncommit: don't allow dirty working copy with PATH (issue5977).
Wed, Mar 6, 12:49 AM
navaneeth.suresh added a comment to D6027: mq: make unshelve to apply on modified mq patch (issue4318).

Ping for review.

Wed, Mar 6, 12:45 AM
navaneeth.suresh updated the diff for D6069: uncommit: add flag --allow-dirty-working-copy.
Wed, Mar 6, 12:35 AM
navaneeth.suresh updated the diff for D5940: uncommit: don't allow dirty working copy with PATH (issue5977).
Wed, Mar 6, 12:35 AM

Mon, Mar 4

navaneeth.suresh added inline comments to D6069: uncommit: add flag --allow-dirty-working-copy.
Mon, Mar 4, 10:40 AM
navaneeth.suresh updated the diff for D6069: uncommit: add flag --allow-dirty-working-copy.
Mon, Mar 4, 10:39 AM
navaneeth.suresh created D6069: uncommit: add flag --allow-dirty-working-copy.
Mon, Mar 4, 10:04 AM
navaneeth.suresh updated the diff for D5940: uncommit: don't allow dirty working copy with PATH (issue5977).
Mon, Mar 4, 9:11 AM
navaneeth.suresh added inline comments to D5940: uncommit: don't allow dirty working copy with PATH (issue5977).
Mon, Mar 4, 9:08 AM

Sun, Mar 3

navaneeth.suresh added inline comments to D5940: uncommit: don't allow dirty working copy with PATH (issue5977).
Sun, Mar 3, 11:11 PM
navaneeth.suresh created D6056: patch: stop aborting when add/rename/copy files on --interactive (issue5727).
Sun, Mar 3, 2:24 PM

Sat, Mar 2

navaneeth.suresh added a comment to D5940: uncommit: don't allow dirty working copy with PATH (issue5977).

I don't understand the commit message. experimental.uncommitondirtydir already works with PATH, doesn't it? I think you mean something like "uncommit: allow dirty working copy with PATH". Did I understand that right?

Sat, Mar 2, 12:35 AM

Fri, Mar 1

navaneeth.suresh added inline comments to D5940: uncommit: don't allow dirty working copy with PATH (issue5977).
Fri, Mar 1, 2:39 PM
navaneeth.suresh updated the diff for D5940: uncommit: don't allow dirty working copy with PATH (issue5977).
Fri, Mar 1, 2:36 PM

Thu, Feb 28

navaneeth.suresh updated the diff for D5940: uncommit: don't allow dirty working copy with PATH (issue5977).
Thu, Feb 28, 11:47 PM

Tue, Feb 26

navaneeth.suresh created D6027: mq: make unshelve to apply on modified mq patch (issue4318).
Tue, Feb 26, 7:24 AM
navaneeth.suresh added a comment to D5940: uncommit: don't allow dirty working copy with PATH (issue5977).

Ping for review.

Tue, Feb 26, 5:59 AM

Mon, Feb 25

navaneeth.suresh added a comment to D5997: mq: disable qrecord during histedit (issue5981).

Thanks for queuing @pulkit. @yuja thanks for the review.

Mon, Feb 25, 10:43 AM
navaneeth.suresh closed D5997: mq: disable qrecord during histedit (issue5981).
Mon, Feb 25, 10:40 AM
navaneeth.suresh committed rHG4d21ebc4cb47: mq: disable qrecord during histedit (issue5981).
mq: disable qrecord during histedit (issue5981)
Mon, Feb 25, 10:40 AM

Sun, Feb 24

navaneeth.suresh updated the diff for D5997: mq: disable qrecord during histedit (issue5981).
Sun, Feb 24, 10:52 PM

Feb 21 2019

navaneeth.suresh created D5997: mq: disable qrecord during histedit (issue5981).
Feb 21 2019, 2:33 PM

Feb 16 2019

navaneeth.suresh updated the diff for D5940: uncommit: don't allow dirty working copy with PATH (issue5977).
Feb 16 2019, 12:26 AM
navaneeth.suresh added inline comments to D5940: uncommit: don't allow dirty working copy with PATH (issue5977).
Feb 16 2019, 12:13 AM
navaneeth.suresh updated the diff for D5940: uncommit: don't allow dirty working copy with PATH (issue5977).
Feb 16 2019, 12:07 AM

Feb 15 2019

navaneeth.suresh added inline comments to D5940: uncommit: don't allow dirty working copy with PATH (issue5977).
Feb 15 2019, 2:31 PM
navaneeth.suresh updated the diff for D5940: uncommit: don't allow dirty working copy with PATH (issue5977).
Feb 15 2019, 2:08 PM
navaneeth.suresh added inline comments to D5940: uncommit: don't allow dirty working copy with PATH (issue5977).
Feb 15 2019, 11:53 AM

Feb 14 2019

navaneeth.suresh committed rHG5fe4de392edb: revset: improve documentation on expectsize().
revset: improve documentation on expectsize()
Feb 14 2019, 4:17 PM
navaneeth.suresh updated the diff for D5940: uncommit: don't allow dirty working copy with PATH (issue5977).
Feb 14 2019, 11:56 AM
navaneeth.suresh added a comment to D5940: uncommit: don't allow dirty working copy with PATH (issue5977).
In D5940#87026, @pulkit wrote:

Hi, it will be good if you specify that the patch is authored by someone else, mention their name and also provide the link to the original PR of this patch.

Feb 14 2019, 9:48 AM
navaneeth.suresh closed D5953: revset: improve documentation on expectsize().
Feb 14 2019, 8:24 AM
navaneeth.suresh committed rHG735da363556d: revset: improve documentation on expectsize().
revset: improve documentation on expectsize()
Feb 14 2019, 8:23 AM

Feb 13 2019

navaneeth.suresh updated the diff for D5953: revset: improve documentation on expectsize().
Feb 13 2019, 8:00 AM
navaneeth.suresh created D5953: revset: improve documentation on expectsize().
Feb 13 2019, 7:51 AM
navaneeth.suresh added inline comments to D5813: revset: add expect to check the size of a set.
Feb 13 2019, 7:27 AM

Feb 12 2019

navaneeth.suresh closed D5813: revset: add expect to check the size of a set.
Feb 12 2019, 7:51 AM
navaneeth.suresh committed rHG8185c8abce87: revset: add expectsize to check the size of a set.
revset: add expectsize to check the size of a set
Feb 12 2019, 7:50 AM

Feb 11 2019

navaneeth.suresh created D5940: uncommit: don't allow dirty working copy with PATH (issue5977).
Feb 11 2019, 2:02 PM
navaneeth.suresh updated the diff for D5813: revset: add expect to check the size of a set.
Feb 11 2019, 10:14 AM

Feb 10 2019

navaneeth.suresh added a comment to D5813: revset: add expect to check the size of a set.

nit-pick is resolved. getintrange() will throw a ParseError on setting size to one of min:, :max or :. In ideal case, on calling from expectsize() it shouldn't fail.

Feb 10 2019, 11:13 PM
navaneeth.suresh updated the diff for D5813: revset: add expect to check the size of a set.
Feb 10 2019, 11:09 PM
navaneeth.suresh updated the diff for D5813: revset: add expect to check the size of a set.
Feb 10 2019, 12:19 AM

Feb 7 2019

navaneeth.suresh added a comment to D5813: revset: add expect to check the size of a set.

@yuja I've updated the revision with the suggested changes.

Feb 7 2019, 2:34 PM
navaneeth.suresh updated the diff for D5813: revset: add expect to check the size of a set.
Feb 7 2019, 2:28 PM
navaneeth.suresh abandoned D5490: commit: remove ignore whitespace option on --interactive (issue6042).

rHG66399f2e92aa solves this issue, many thanks @spectral. Closing this.

Feb 7 2019, 11:46 AM

Feb 6 2019

navaneeth.suresh updated subscribers of D5490: commit: remove ignore whitespace option on --interactive (issue6042).

@spectral @yuja Can I close this?

Feb 6 2019, 11:26 AM
navaneeth.suresh added a comment to D5813: revset: add expect to check the size of a set.

Can you share your failed attempt?

Maybe you can get how revset works by testing expression with
hg debugrevspec -v.

Feb 6 2019, 11:19 AM
navaneeth.suresh updated the diff for D5813: revset: add expect to check the size of a set.
Feb 6 2019, 11:16 AM

Feb 5 2019

navaneeth.suresh added a comment to D5813: revset: add expect to check the size of a set.

First, I think the word expect is too general. Perhaps, this should be called
expectsize() or expectlen().

Feb 5 2019, 12:33 PM

Feb 4 2019

navaneeth.suresh committed rHG09f1c17e24dd: tags: avoid generating commit for an already deleted tag (issue5752).
tags: avoid generating commit for an already deleted tag (issue5752)
Feb 4 2019, 4:54 PM
navaneeth.suresh updated the diff for D5813: revset: add expect to check the size of a set.
Feb 4 2019, 1:19 AM
navaneeth.suresh added a comment to D5813: revset: add expect to check the size of a set.

@pulkit Thanks for the quick review! I've updated the revision with the suggested changes. @durin42 It would be great if you let us know your opinion on one(<set>) as an alias for expect(<set>, 1). I did it as it was present in WeShouldDoThat page.

Feb 4 2019, 1:07 AM
navaneeth.suresh updated the diff for D5813: revset: add expect to check the size of a set.
Feb 4 2019, 1:03 AM

Feb 3 2019

navaneeth.suresh created D5813: revset: add expect to check the size of a set.
Feb 3 2019, 9:28 AM

Jan 31 2019

navaneeth.suresh closed D5780: tags: avoid generating commit for an already deleted tag (issue5752).
Jan 31 2019, 4:20 PM
navaneeth.suresh committed rHG8db09278f7e0: tags: avoid generating commit for an already deleted tag (issue5752).
tags: avoid generating commit for an already deleted tag (issue5752)
Jan 31 2019, 4:19 PM
navaneeth.suresh added a comment to D5780: tags: avoid generating commit for an already deleted tag (issue5752).

@durin42 Thanks for the quick review.
@mharbison72 Yup. I should not be making changes to the ui as this can be fixed simply within the call itself. My bad. I've updated the revision as per your suggestion, many thanks!

Jan 31 2019, 2:28 PM
navaneeth.suresh updated the diff for D5780: tags: avoid generating commit for an already deleted tag (issue5752).
Jan 31 2019, 2:21 PM
navaneeth.suresh created D5780: tags: avoid generating commit for an already deleted tag (issue5752).
Jan 31 2019, 8:30 AM
navaneeth.suresh added a comment to D5744: commit: ignore diff whitespace settings when doing `commit -i` (issue5839).

Sorry for bumping this up. I worked on a similar issue on D5490. My initial revision was similar to this one. @spectral Don't you think it'll affect record extension which was written with zero test coverage? Please don't feel obligated. I am just being curious.

Jan 31 2019, 12:59 AM

Jan 26 2019

navaneeth.suresh closed D5709: diffstat: support filenames with whitespaces on renames.
Jan 26 2019, 4:31 AM
navaneeth.suresh committed rHG4a33a6bf2b52: diffstat: support filenames with whitespaces on renames.
diffstat: support filenames with whitespaces on renames
Jan 26 2019, 4:30 AM
navaneeth.suresh created D5709: diffstat: support filenames with whitespaces on renames.
Jan 26 2019, 3:26 AM
navaneeth.suresh committed rHGf1b0d9988825: diffstat: make --git work properly on renames (issue6025).
diffstat: make --git work properly on renames (issue6025)
Jan 26 2019, 2:09 AM
navaneeth.suresh added a comment to D5628: diffstat: make --git work properly on renames (issue6025).

One nit: better to not use .split() since a filename may include whitespace.

Jan 26 2019, 1:50 AM

Jan 25 2019

navaneeth.suresh added a comment to D5628: diffstat: make --git work properly on renames (issue6025).

Thanks for queuing this!

Jan 25 2019, 9:53 PM
navaneeth.suresh closed D5628: diffstat: make --git work properly on renames (issue6025).
Jan 25 2019, 3:11 PM
navaneeth.suresh committed rHGffdac0067a14: diffstat: make --git work properly on renames (issue6025).
diffstat: make --git work properly on renames (issue6025)
Jan 25 2019, 3:11 PM

Jan 17 2019

navaneeth.suresh created D5628: diffstat: make --git work properly on renames (issue6025).
Jan 17 2019, 7:46 AM

Jan 13 2019

navaneeth.suresh added a comment to D5490: commit: remove ignore whitespace option on --interactive (issue6042).

_performrevert() would be in the same boat, but it was explicitly flagged on
at f37a69ec3f47. This implies that the diff.ignorews option would be used
in practice to exclude whitespace changes while interactive commit/revert.
So disabling any whitespace options would break someone's workflow.

Jan 13 2019, 9:31 AM

Jan 12 2019

navaneeth.suresh closed D5552: tests: add test for warning on histedit with tagged commits.
Jan 12 2019, 9:08 PM
navaneeth.suresh committed rHG172296c6db91: tests: add test for warning on histedit with tagged commits.
tests: add test for warning on histedit with tagged commits
Jan 12 2019, 9:08 PM

Jan 11 2019

navaneeth.suresh added a comment to D5490: commit: remove ignore whitespace option on --interactive (issue6042).

Sorry for late, but doesn't it break hg record --ignore-all-space, etc.?

Jan 11 2019, 10:13 AM
navaneeth.suresh updated the diff for D5490: commit: remove ignore whitespace option on --interactive (issue6042).
Jan 11 2019, 10:12 AM

Jan 10 2019

navaneeth.suresh added a comment to D5490: commit: remove ignore whitespace option on --interactive (issue6042).

@durin42 I've added test for the current revision. Unfortunately, I'm unable to write a test for the wrong behaviour. I'll try writing it again. Any hints from your side?

Jan 10 2019, 11:29 PM
navaneeth.suresh updated the diff for D5490: commit: remove ignore whitespace option on --interactive (issue6042).
Jan 10 2019, 11:26 PM
navaneeth.suresh added a comment to D5490: commit: remove ignore whitespace option on --interactive (issue6042).

Could you add a test?

(Bonus points if you add a test that shows the wrong behavior and then fix it in this change.)

Jan 10 2019, 8:13 AM
navaneeth.suresh created D5552: tests: add test for warning on histedit with tagged commits.
Jan 10 2019, 8:10 AM
navaneeth.suresh added a comment to D5494: histedit: add user input to warning message on editing tagged commits.
In D5494#81855, @pulkit wrote:

Sorry for bumping on this now, can you add tests for this as follow-up?

Jan 10 2019, 3:25 AM

Jan 9 2019

navaneeth.suresh committed rHG7b7e081f8954: histedit: add user input to warning message on editing tagged commits.
histedit: add user input to warning message on editing tagged commits
Jan 9 2019, 3:00 PM
navaneeth.suresh closed D5494: histedit: add user input to warning message on editing tagged commits.
Jan 9 2019, 3:00 PM

Jan 7 2019

navaneeth.suresh added a comment to D5503: vfs: add support for repo names with `$` when using with env vars (issue5739).

This is logically incorrect. The problem is that we're doing variable
expansion at too lower layer. vfs(expand(user_specified_path)) makes
some sense, but vfs(expand(getcwd())) is clearly wrong. And the vfs class
can't know where the base comes from.

Jan 7 2019, 11:08 PM
navaneeth.suresh updated the diff for D5503: vfs: add support for repo names with `$` when using with env vars (issue5739).
Jan 7 2019, 7:08 AM
navaneeth.suresh created D5503: vfs: add support for repo names with `$` when using with env vars (issue5739).
Jan 7 2019, 3:38 AM