Skip to content

Conversation

@Onder7994
Copy link

@Onder7994 Onder7994 commented Nov 24, 2025

Hello! I add dependency control via poetry. This is more convenient. And any can install code like this

pip install "git+https://github.com/vladko312/SSTImap.git#branch=master"

And code will installed with entrypoint script - sstimap.

You can check it

poetry install
poetry run sstimap

Build and install localy

poetry install
poetry build
pip install dist/sstimap-1.3.1-py3-none-any.whl
sstimap

Chek on my fork repo

python3 -m venv .sstimap
source .sstimap/bin/activate
pip install "git+https://github.com/Onder7994/SSTImap.git@feature/pyproject"
sstimap -h

@vladko312
Copy link
Owner

This is a good idea! I also considered making a python module, but with a way to use both as a proper module and as a tool.
I delayed that update, as I want to make some changes to the architecture before that.

For now, I will not merge this request, as it has some issues. For example, it ONLY works as module, as it obtains the version info from the module. This will prevent SSTImap from being used as a standalone tool, as well as installing multiple versions at the same time.
I might merge it later after fixing some things and changing the architecture.

Also, can you provide more details about other changes? E.g. why os-based file and directory access was replaced by pathlib?

@vladko312 vladko312 added enhancement New feature or request question Further information is requested delayed The issue will be fixed with a big update later labels Nov 25, 2025
@Onder7994
Copy link
Author

Thank you for you answer!

I move to pathlib, because this is more convenient with poetry. Old way with sys.path does't work any more, for my opinion pathlib versatile and easier.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

delayed The issue will be fixed with a big update later enhancement New feature or request question Further information is requested

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants