search

Nama / swayrst

Restore workspaces in sway to displays and move applications to saved workspaces
MIT License
54 stars 7 forks source link

Use setup.py scripts option so that the sway-workspaces script can be installed #7

Closed jsimonrichard closed 1 year ago

jsimonrichard commented 1 year ago

I haven't figured out how to get this script working on my system yet (there's probably something I'm misunderstanding), but your usage of setup.py didn't make a whole lot of sense to me. So, I made this PR on the chance that might be helpful.

This PR makes it a lot easier to install the script using pip (just run pip install .).

Kindly disregard this PR if this isn't what you intended for this project.

Nama commented 1 year ago

I can't merge this PR. I don't understand your commit to the README.md, The changes don't really make sense to me. Your replaced packages in the setup.py instead of just adding scripts. Also, the whole project uses single quotes, lets keep it that way ;)

I just pushed a commit, which should make the package available on the cli. If not, open an issue so we can have a look again.