Skip to content

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