search

danielcopper / wezterm-session-manager

MIT License
83 stars 16 forks source link

Saving workspace state in a single file #8

Open zDonik1 opened 6 months ago

zDonik1 commented 6 months ago

What was the reasoning behind saving each workspace in a separate file instead of storing it all in a single file?

There are 2 issues that I have encountered that come off of this structure.

  1. Non-standard workspace names that have the "/" or "\" characters can't be opened.
  2. Finding all the saved workspaces requires using LuaFS to find the files of the saved states. If the state was saved in a single file we could get the entire list of saved workspace states.

Will it be possible to change the structure to save the state in a single file?

danielcopper commented 6 months ago

There is no real reason why it's multiple files. These are good points. I am planning on cleaning and updating the script currently and will include this.

But feel free to do a pr if you don't want to wait a bit.