-
Notifications
You must be signed in to change notification settings - Fork 51
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
Feat cml tmux #143
Feat cml tmux #143
Conversation
This commit introduce a new command 'cml tmux' It allows users to connect to multiple node consoles using 'tmux'. It creates a new 'tmux' session with the lab title and the first 4 characters of the lab ID. The `--group` option offers two grouping choices: - 'panes': Group all connections within a single 'tmux' window. (default) - 'windows': Represent each connection as an individual 'tmux' window. Version 2 only.
Pull Request Test Coverage Report for Build 6700857795
💛 - Coveralls |
@@ -1 +1 @@ | |||
__version__ = "1.4.2" | |||
__version__ = "1.4.3" |
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 do not bump version yourself.
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.
Sorry, I based my branch out of master
, do you want me to change it to 1.4.2?
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.
joe-dev should have 1.4.3. I see it in my pull.
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.
So we good?
@@ -7,6 +7,7 @@ gunicorn | |||
idna | |||
itsdangerous | |||
Jinja2 | |||
libtmux |
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.
Is this also required for testing?
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.
right :)!
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.
Then it needs to be in test-requirements.txt.
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.
Committed already!
2b93385
Since we are it, time to stop procrastinating and submit that PR for tmux support :)