-
Notifications
You must be signed in to change notification settings - Fork 232
Add ls option to cli #599
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 ls option to cli #599
Conversation
Nice @pythops ! (Also nice username!) Can you add a test? (e.g. https://github.com/tmux-python/tmuxp/blob/master/tests/test_cli.py)? |
Thanks @tony :) |
@pythops I will take a look at this tomorrow I need to take a look at why Travis isn't triggering. |
I will take a peek tomorrow. It looks like there's another error happening at the same time not related to this PR. |
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.
Looks good! Some change requests
Additionally, can you do a rebase against master so the latest fixes for travis CI are in?
I'm not sure what your repo layout is like (git remotes) but it's something like this:
git remote -v
Assuming your origin is:
origin https://github.com/tmux-python/tmuxp.git (fetch)
origin https://github.com/tmux-python/tmuxp.git (push)
git pull --rebase origin master
git push --force-with-lease
If origin
is your own branch, git remote add upstream https://github.com/tmux-python/tmuxp.git
and do this:
git pull --rebase upstream master
git push --force-with-lease
@tony Would you mind restart the tests, they should pass. |
@pythops Restarted |
@pythops You can also rebase again - I did several fixes to travis since you last pushed |
The CI failed because of poetry. Maybe it should be restarted again as I did the rebase after. |
You need to rebase the against my master branch - which has poetry - and force push it. It's not easy to give instructions on since I don't know how you have Side note: It's really difficult to find a tutorial online that's satisfactory for me to explain - since it really hinges on your local github layout and how you have When you rebase your branch against my master branch at Can you give me access to your forked repo? I can do it for you |
@pythops Hm, try this:
This assures that:
You should see an option to open a new pull request, go ahead and do it if you'd like Now, to rebase so your work is based on the latest tmuxp:
|
@tony you should have access to my fork |
Sorry about that, not sure why the branch closed |
@pythops In what sense do I have access to your repo? I don't seem to be able to push branches See #616, I PR'd it, but the problem is - I'd sort of take credit for your work. I want to make sure you get credit for the Pull request/commits. I think you need to redo it and rebase correctly - it's really up to your system and reading up on rebasing, I wrote more here: #599 (comment) Do you want to make your own pull request? If so, I think you need to figure out the branching and make a new PR. If I do it for you, I end up becoming the creator of the PR and part of the commit messages. |
Also sorry for all the trouble with this PR! Initially our CI system was broke. And rebasing it kind of tricky to explain without me knowing your configuration. If I gave instructions and the remote names didn't match, the instructions wouldn't work. |
Almost missed this! Thank you for #599
No description provided.