Skip to content
This repository has been archived by the owner on Dec 21, 2022. It is now read-only.

Remove storm-web from the codebase #137

Open
emre opened this issue Feb 13, 2017 · 2 comments
Open

Remove storm-web from the codebase #137

emre opened this issue Feb 13, 2017 · 2 comments

Comments

@emre
Copy link
Owner

emre commented Feb 13, 2017

Maintaining it is hard. People want to use it can make a separated package and maintain it.

@jaddison
Copy link

jaddison commented Jun 8, 2017

If removing it isn't an option, it would be good to have it as an optional package. So:

pip install stormssh

would install the cmd-line only storm, while:

pip install stormssh[web]

would install storm along with web components, giving storm web functionality.

Not having fully ready the docs, I was surprised to see flask being installed just so I could manage my ssh connections! :)

@denzuko
Copy link

denzuko commented Dec 26, 2017

@emre sure, I'll be glad to maintain it. Even submitted an PR with a few changes to make the web part easier to maintain and use with extra systems.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants