Skip to content

Feature: allow setting up custom environment when creating windows and/or panes #453

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

Conversation

zappolowski
Copy link
Contributor

This is required for implementing tmux-python/tmuxp#832.

@tony
Copy link
Member

tony commented Nov 1, 2022

@zappolowski FYI on timeline, I may not be able to give this my undivided attention until the evening (weekends latest)

@zappolowski zappolowski force-pushed the feature/set-environment-for-windows-and-panes branch from e67fb47 to 3576909 Compare November 2, 2022 05:02
@tony tony force-pushed the feature/set-environment-for-windows-and-panes branch 2 times, most recently from 2295b53 to 9ddfbb8 Compare November 5, 2022 13:55
@codecov
Copy link

codecov bot commented Nov 5, 2022

Codecov Report

Merging #453 (fca7c0d) into master (c425c37) will increase coverage by 0.22%.
The diff coverage is 96.07%.

@@            Coverage Diff             @@
##           master     #453      +/-   ##
==========================================
+ Coverage   87.46%   87.68%   +0.22%     
==========================================
  Files          20       20              
  Lines        1803     1852      +49     
  Branches      280      298      +18     
==========================================
+ Hits         1577     1624      +47     
  Misses        154      154              
- Partials       72       74       +2     
Impacted Files Coverage Δ
tests/test_session.py 97.05% <94.73%> (-0.31%) ⬇️
tests/test_window.py 97.59% <95.23%> (-0.27%) ⬇️
src/libtmux/session.py 77.09% <100.00%> (+0.65%) ⬆️
src/libtmux/window.py 78.73% <100.00%> (+0.62%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@tony tony self-requested a review November 5, 2022 14:04
@tony tony force-pushed the feature/set-environment-for-windows-and-panes branch 2 times, most recently from 9ddfbb8 to 6c0457f Compare November 5, 2022 14:50
@tony tony self-requested a review November 5, 2022 17:54
tony
tony previously approved these changes Nov 5, 2022
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.

LGTM

Fine to squash the commits as you see fit

@zappolowski zappolowski force-pushed the feature/set-environment-for-windows-and-panes branch from 8e24396 to 9016222 Compare November 5, 2022 18:21
@tony
Copy link
Member

tony commented Nov 5, 2022

@zappolowski Game plan

I will merge #455 and make a patch release for 0.15.9 0.15.10

we can then merge this (i will rebase) and do final check, if it looks good, will release 0.16.0

@tony tony force-pushed the feature/set-environment-for-windows-and-panes branch from 9016222 to 214c692 Compare November 5, 2022 18:44
@tony
Copy link
Member

tony commented Nov 5, 2022

@tony tony self-requested a review November 6, 2022 14:11
This allows to set up a custom environment when creating a new window.
This allows to set up a custom environment when creating a new pane.
@zappolowski zappolowski force-pushed the feature/set-environment-for-windows-and-panes branch from 0f293af to 4244049 Compare November 6, 2022 17:28
@tony
Copy link
Member

tony commented Nov 6, 2022

@zappolowski I didn't get a notification of the push! Merging and publishing now

@tony tony merged commit 6a03ad9 into tmux-python:master Nov 6, 2022
tony added a commit to tmux-python/tmuxp that referenced this pull request Nov 6, 2022
Passthrough of environmental variables via -e in tmux 3.0 and above.

See also: tmux-python/libtmux#453
@tony
Copy link
Member

tony commented Nov 6, 2022

@zappolowski

The floor is yours to address tmux-python/tmuxp#832 (comment)

@zappolowski zappolowski deleted the feature/set-environment-for-windows-and-panes branch November 7, 2022 16:16
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