Skip to content
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

Fix to tmuxp freeze command #491

Merged
merged 1 commit into from May 27, 2019

Conversation

Projects
None yet
3 participants
@kstilwell
Copy link
Contributor

commented May 17, 2019

If no session name is passed to tmuxp freeze, use what is most likely the current session as the default value. This matches the current documentation for this command.

Meant to address #485

- Allow for session name to not be passed to `tmux freeze`
- If session is not provided grab what is most likely the current session

@kstilwell kstilwell requested a review from tony May 17, 2019

@codecov-io

This comment has been minimized.

Copy link

commented May 17, 2019

Codecov Report

Merging #491 into master will decrease coverage by 0.18%.
The diff coverage is 25%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #491      +/-   ##
==========================================
- Coverage   75.56%   75.37%   -0.19%     
==========================================
  Files           5        5              
  Lines         794      796       +2     
  Branches      231      232       +1     
==========================================
  Hits          600      600              
- Misses        144      146       +2     
  Partials       50       50
Impacted Files Coverage Δ
tmuxp/cli.py 56.49% <25%> (-0.35%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b90f7ec...ee8e167. Read the comment docs.

@kstilwell kstilwell force-pushed the issue_485_work branch from 418ab9f to ee8e167 May 26, 2019

@tony

This comment has been minimized.

Copy link
Member

commented May 27, 2019

@kstilwell Makes sense.

@tony tony merged commit 96fddc1 into master May 27, 2019

3 of 5 checks passed

codecov/patch 25% of diff hit (target 75.56%)
Details
codecov/project 75.37% (-0.19%) compared to b90f7ec
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
pyup.io/safety-ci No dependencies with known security vulnerabilities.
Details

@tony tony deleted the issue_485_work branch May 27, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.