-
Notifications
You must be signed in to change notification settings - Fork 229
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
Pass start_directory to new_session when building a session #829
Conversation
This fixes a bug introduced in 2a4714e76b3a85b3391b05413f36623bcb1493f9 (#809) whereby the `start_directory` session config was no longer passed to the `new_session` method when building a session. This resulted in the session working directory is not being set correctly, so new windows/panes created in the session, would not start in the expected directory.
Codecov Report
@@ Coverage Diff @@
## master #829 +/- ##
=======================================
Coverage 75.00% 75.00%
=======================================
Files 18 18
Lines 1396 1396
Branches 326 326
=======================================
Hits 1047 1047
Misses 263 263
Partials 86 86
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
@heindsight Very good catch. Would you like to add anything to |
@tony thanks. I don't think this really needs to be mentioned in |
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, thank you for the test!
@heindsight This is live in 1.15.3 If you try it out, any better? |
@tony Yes, that's much better. Thanks 😃 |
@heindsight Excellent! |
This fixes a bug introduced in 2a4714e76b3a85b3391b05413f36623bcb1493f9 (#809) whereby the
start_directory
session config was no longer passed to thenew_session
method when building a session. This resulted in the session working directory is not being set correctly, so new windows/panes created in the session, would not start in the expected directory.