Skip to content

modules/tmux: init#89

Closed
skyler-oakeson wants to merge 1 commit into
Lassulus:mainfrom
skyler-oakeson:tmux
Closed

modules/tmux: init#89
skyler-oakeson wants to merge 1 commit into
Lassulus:mainfrom
skyler-oakeson:tmux

Conversation

@skyler-oakeson

Copy link
Copy Markdown
Contributor

This adds a simple tmux.conf option.

@zimward zimward left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

looks pretty good otherwise. please squash your commits into a single one named modules/tmux: init once you think its ready

Comment thread modules/tmux/check.nix Outdated
Comment thread modules/tmux/check.nix Outdated
Comment thread modules/tmux/module.nix Outdated
@skyler-oakeson skyler-oakeson force-pushed the tmux branch 3 times, most recently from daccd19 to 39a21f1 Compare January 22, 2026 06:09
@skyler-oakeson skyler-oakeson changed the title Tmux modules/tmux: init Jan 22, 2026
@zimward

zimward commented Jan 22, 2026

Copy link
Copy Markdown
Collaborator

seems like the test fails due to tmux not finding a socket or smth

@zimward

zimward commented Jan 22, 2026

Copy link
Copy Markdown
Collaborator

if you want to debug this nix flake check does run the tests locally

@a-kenji

a-kenji commented Mar 31, 2026

Copy link
Copy Markdown
Contributor

Hey! What is the status of this wrapper module?

@skyler-oakeson

Copy link
Copy Markdown
Contributor Author

Sorry life got busy and I spaced this. I'll try to get to this on the weekend.

@a-kenji

a-kenji commented Mar 31, 2026

Copy link
Copy Markdown
Contributor

No worries. :). Thank you.

@ItsLiyua

Copy link
Copy Markdown

Can you pull the latest dev/master/main/whatever branch into this please? I'm using this PR branch and it still depends on the replace package which is unfree. This was fixed at some point but this PR doesn't have the fix yet.

@skyler-oakeson

Copy link
Copy Markdown
Contributor Author

Can you pull the latest dev/master/main/whatever branch into this please? I'm using this PR branch and it still depends on the replace package which is unfree. This was fixed at some point but this PR doesn't have the fix yet.

The branch has been updated.

@skyler-oakeson

Copy link
Copy Markdown
Contributor Author

I have closed this one to fix the commit history so that the other one contains only one commit. I am sorry about the inconvenience.

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.

4 participants