-
Notifications
You must be signed in to change notification settings - Fork 5
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add command history #79
Add command history #79
Conversation
- Implemented storage for the latest typed command - Fix bug where certain lines are iterated through multiple times
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
err
resolved @koh-jx |
Eh wait havent commit yet LOL |
@yeppog Is this ready for review? |
…into add-command-history
…into add-command-history
…into add-command-history
…into add-command-history
30ddc0d
to
9732dca
Compare
Codecov Report
@@ Coverage Diff @@
## master #79 +/- ##
============================================
+ Coverage 57.30% 58.14% +0.83%
- Complexity 555 563 +8
============================================
Files 110 110
Lines 2342 2377 +35
Branches 225 229 +4
============================================
+ Hits 1342 1382 +40
+ Misses 921 916 -5
Partials 79 79
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Allow for filtering through previously executed commands through the CLI through the arrow keys (up and down) as navigation.
Closes #64 .