Skip to content
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.

Sign up for GitHub

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

Jump to bottom

fix: list output with --hide-attached flag #161

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

fix: list output with --hide-attached flag #161

wants to merge 4 commits into from

Conversation

albertilagan
Copy link
Contributor

@albertilagan albertilagan commented Sep 6, 2024 •
edited
Loading

Fixes #160

image

hide currently active/attached session from list

  • when doing listConfig check if there's a tmux session that of the same name/key
  • exclude Attached = 1 from list when --hide-attached flag is present

@joshmedeski
Copy link
Owner

joshmedeski commented Sep 6, 2024 •
edited
Loading

I manually removed the --hide-attached feature because it's not working correctly.

1586ea6

@joshmedeski joshmedeski closed this Sep 6, 2024
@albertilagan
Copy link
Contributor Author

I manually removed the --hide-attached feature because it's not working correctly.

1586ea6

latest on my branch should fix your issue, thanks for quick update though :)

@joshmedeski
Copy link
Owner

Thanks @albertilagan I'll reopen this and test it.

@joshmedeski joshmedeski reopened this Sep 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Session from config not showing up
2 participants
@albertilagan @joshmedeski