-
Notifications
You must be signed in to change notification settings - Fork 13
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
added session toolbar #217
Conversation
mikethebeer
commented
Aug 9, 2017
- shows the current session user
- shows the handler nodes connected with
Codecov Report
@@ Coverage Diff @@
## master #217 +/- ##
==========================================
- Coverage 83.05% 82.28% -0.78%
==========================================
Files 16 17 +1
Lines 2066 2105 +39
==========================================
+ Hits 1716 1732 +16
- Misses 350 373 +23
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
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.
When I use this branch and connect to a host that doesn't exist I get repeatedly CONNECT ERROR printed out as soon as I type something?
src/crate/crash/layout.py
Outdated
'create_layout', | ||
) | ||
|
||
def create_layout(message='', lexer=None, |
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.
please add a comment explaining the difference to create_prompt_layout
src/crate/crash/layout.py
Outdated
from prompt_toolkit.layout.margins import PromptMargin, ConditionalMargin | ||
from prompt_toolkit.layout.utils import token_list_width | ||
|
||
__all__ = ( |
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.
no need to define __all__
as we don't do a import *
src/crate/crash/repl.py
Outdated
if cmd.is_conn_avaliable(): | ||
active_servers = cmd.connection.client.active_servers | ||
active_servers = [n.replace('http://', '') for n in active_servers] | ||
active_servers = [n.replace('https://', '') for n in active_servers] |
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.
could use generator expressions instead of list comprehensions to avoid the copying
4 similar comments
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
* shows the current session user * shows the handler nodes connected with