Skip to content

Bug start directory #639

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

Merged
merged 4 commits into from
Nov 1, 2020
Merged

Bug start directory #639

merged 4 commits into from
Nov 1, 2020

Conversation

joseph-flinn
Copy link
Contributor

start_directory was not being passed through when creating a new session.

Issue: #631

@joseph-flinn joseph-flinn requested a review from tony October 31, 2020 23:45
@joseph-flinn joseph-flinn self-assigned this Oct 31, 2020
@@ -1,6 +1,6 @@
__title__ = 'tmuxp'
__package_name__ = 'tmuxp'
__version__ = '1.5.7post0'
__version__ = '1.5.7post1'
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@joseph-flinn Can you make this 1.5.8?

Suggested change
__version__ = '1.5.7post1'
__version__ = '1.5.8'

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yep

@codecov
Copy link

codecov bot commented Oct 31, 2020

Codecov Report

Merging #639 into master will decrease coverage by 0.18%.
The diff coverage is 33.33%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #639      +/-   ##
==========================================
- Coverage   78.14%   77.95%   -0.19%     
==========================================
  Files           6        6              
  Lines         828      830       +2     
  Branches      239      240       +1     
==========================================
  Hits          647      647              
- Misses        121      122       +1     
- Partials       60       61       +1     
Impacted Files Coverage Δ
tmuxp/workspacebuilder.py 87.42% <33.33%> (-1.06%) ⬇️

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 6469857...086e118. Read the comment docs.

Copy link
Member

@tony tony left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@joseph-flinn Make it 1.5.8 and update CHANGES - I will publish it now 😊

@joseph-flinn joseph-flinn removed their assignment Oct 31, 2020
session = self.server.new_session(
session_name=self.sconf['session_name'],
start_directory=self.sconf['start_directory'],
)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's late so I don't want to dredge this up, do we want/need a test for this?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm happy merging without a test, I can write one later if we need it

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No worries. It would probably be a good idea to write a test just so we have it

@tony tony merged commit 3f5fdd6 into master Nov 1, 2020
@tony tony deleted the bug-start-directory branch November 1, 2020 00:15
@tony
Copy link
Member

tony commented Nov 1, 2020

@joseph-flinn v1.5.8 live

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants