Skip to content

Added recursive print depth meta command.#85

Closed
perovic wants to merge 2 commits into
root-project:masterfrom
perovic:RecursivePrintDepthMetaCommand
Closed

Added recursive print depth meta command.#85
perovic wants to merge 2 commits into
root-project:masterfrom
perovic:RecursivePrintDepthMetaCommand

Conversation

@perovic
Copy link
Copy Markdown
Contributor

@perovic perovic commented Sep 8, 2015

Implemented corresponding print behavior.
Maybe revise this meta command's behavior in the case of a missing argument (print out the current value, instead of toggling between two defaults), if it turns out to be more intuitive.

@perovic perovic force-pushed the RecursivePrintDepthMetaCommand branch from e7a17d5 to e0ff729 Compare September 9, 2015 13:47
Implemented corresponding print behaviour.
@perovic perovic force-pushed the RecursivePrintDepthMetaCommand branch from e0ff729 to 68218bc Compare September 14, 2015 05:56
@Axel-Naumann Axel-Naumann self-assigned this Dec 11, 2015
@vgvassilev
Copy link
Copy Markdown
Member

Not sure about the rationale here but could you rebase and add a test case for this functionality?

@phsft-bot
Copy link
Copy Markdown

Can one of the admins verify this patch?

@vgvassilev
Copy link
Copy Markdown
Member

@Axel-Naumann, shall we close this due to inactivity?

@etejedor
Copy link
Copy Markdown
Contributor

@Axel-Naumann what should we do with this one?

@phsft-bot
Copy link
Copy Markdown

Can one of the admins verify this patch?

@etejedor
Copy link
Copy Markdown
Contributor

@Axel-Naumann should we close this one?

@etejedor
Copy link
Copy Markdown
Contributor

At the risk of being repetitive 😁 @Axel-Naumann can we close this one?

@Axel-Naumann
Copy link
Copy Markdown
Member

Let's close - we keep the code in this PR, but for now there has not been a lot of demand for the feature :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants