Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Convert to module #153

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

Conversation

pombaer
Copy link

@pombaer pombaer commented Jan 27, 2025

No description provided.

@bb-Ricardo
Copy link
Owner

Hey,

Thank you again for this merge request. But after thinking about it for a while it appears that just renaming it is not enough in order to publish this as a library to use.

  • It misses clear interfaces and config options (instantiating only by passing the cli args)
  • the internal functions are not well documented
  • the session handling is not designed to behave as a library

The idea is great but would need quite some refactoring

@pombaer
Copy link
Author

pombaer commented Feb 11, 2025

Ich habe das lt. der Anleitung im README_POETRY.md bei mir packetiert und auf einen Repository Server (nexus) hochgeladen und auch auf einigen Servern laufen, wie gesagt ist es nicht perfekt aber grundsätzlich einmal funktionsfähig und auch ohne breaking changes, was für mich in dem Fall einmal am wichtigsten war, da ich kein Entwickler bin und da nicht allzuviel anpassen wollte.
Also von mir aus kann ich auch damit leben wenn ich die Anpassungen im Falle eines Updates wieder wiederhofen muss, ich würde das weitere Vorgehen zu dem PR bei dir und den anderen Maintainern sehen.

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