Deleted hanging prompt_manager.py module since no file was using it.#864
Deleted hanging prompt_manager.py module since no file was using it.#864ElGreKost wants to merge 3 commits intomodelcontextprotocol:mainfrom
prompt_manager.py module since no file was using it.#864Conversation
prompt_manager.py module since no file was using it.
Kludex
left a comment
There was a problem hiding this comment.
I think this is a file by itself? https://github.com/jlowin/fastmcp/blob/97ef80c62478d0f366d62baf3e15875937cd7b9f/src/fastmcp/prompts/prompt_manager.py#L5
|
Not in the modelcontextprotocol:main where I PR. You show me a different fork from jlowin In the main repo in prompts folder There are the And what I'm saying is they both implement class PromptManger but the And that's why this should be deleted. (and to be consistent with the tools directory Maybe even rename the manager to prompt_manager like the tool_manager in tools |
yes, I think that would be great (I can't directly push to this PR, but happy to take commits and create a new PR with renaming, or please apply the changes and we can merge) |
|
This was already merged, I think. |
Deleted the file prompt_manager.py since now all the files in the directory use the improved version of it that is manager.py