list command was redundant with history command#115
Merged
tleonhardt merged 4 commits intomasterfrom Jun 3, 2017
Merged
Conversation
The extra functionality of listing a span (slice) of history items present in the list command has been incorporated into the existing history command and the list command has been removed. Unit tests have been updated accordingly. In the process a bug was fixed that was causing a crash if and end was supplied to the span but not a start.
Updated these to reflect that the list command has been removed
It no longer displays settable parameters since this functionality was redundant since those are displayed with either set or show. It now shows additional global settings for parsing @options commands. In the future a few more things should probably be displayed here related to parameters which are not settable at runtime.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The list command has been removed since it did pretty much the same thing as the history command.
The feature from the list command to display a "span" of history items ranging from a start index to an end index has been added to the history command. The history command already supported all other features of the list command.
In the process, a bug which caused a crash when specifying an end index but not a start index for the span has been fixed.
Help, documentation, and unit tests have been updated accordingly.
Also made some minor updates to the cmdenvironment command and updated the CHANGES.md file.
This closes #112