Skip to content

Conversation

iznoel
Copy link

@iznoel iznoel commented Aug 18, 2025

This is a contribution to tmuxinator, cc @hschne, following issue 962

In essence, Current schema does not recognize named windows / panes, only anonymous ones
This branch attempts to

  • generalizing the Window and Pane objects
  • introduce their discovery with named objects
    {windows: [{$wname: <window>{panes: [{$pname: <pane>}, ...]}}, ...]}
  • versus the current schema that's using
    { windows: [ <window>{panes: <pane>{}}] }

Current tmuxinator testsuite already uses this configuration variant
But this setup should allow for completion to work correctly aswell

Copy link
Contributor

Thanks for the PR!

This section of the codebase is owned by @madskristensen and @hyperupcall - if they write a comment saying "LGTM" then it will be merged.

@iznoel
Copy link
Author

iznoel commented Aug 18, 2025

so src/test/tmuxinator/sample.deprecations.yml fails, a config like doesn't appear valid for tmuxinator v3.3.5 anyhow 🤷

@hyperupcall
Copy link
Member

Feel free to remove anything that's invalid if you're absolutely sure, once CI goes green I can merge!

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