Skip to content

Support for sessions to save all workspaces#131

Open
DogeTheBeast wants to merge 4 commits intoJonnyHaystack:masterfrom
DogeTheBeast:master
Open

Support for sessions to save all workspaces#131
DogeTheBeast wants to merge 4 commits intoJonnyHaystack:masterfrom
DogeTheBeast:master

Conversation

@DogeTheBeast
Copy link

Added support to save all workspaces using one command. This change uses the flag --session or -S with a session name to store the session. The session is stored in a folder structure shown below:-
image
Where "Default" is the name of the session.

Let me know if I need to change anything.

DogeTheBeast and others added 2 commits December 11, 2025 05:53
* Basic implementation of sessions

* Support for ls and delete and cleanup
@artiekra
Copy link

this is amazing! exactly what i need
its sad that i have to install this manually and its not merged into the main repo

@JonnyHaystack
Copy link
Owner

@DogeTheBeast Thanks for this contribution! Would you mind moving the kitty stuff to a separate branch, as it's now showing up in this PR?

@DogeTheBeast
Copy link
Author

@JonnyHaystack Should be good for review now. Ill make a different PR for the kitty stuff. Lemme know if there is an issue.

Copy link
Owner

@JonnyHaystack JonnyHaystack left a comment

Choose a reason for hiding this comment

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

Thanks so much for the contribution and for your patience! This looks great to me overall. I have left some (pretty minor) comments for when you have a moment to look over them.

import click
import i3ipc
import shutil
from natsort import natsorted
Copy link
Owner

Choose a reason for hiding this comment

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

Should be able to remove this import now

Copy link
Author

Choose a reason for hiding this comment

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

I am guessing you are referring to the natsort. I shall get rid of that

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.

3 participants