Skip to content

Conversation

@nurikk
Copy link

@nurikk nurikk commented Oct 4, 2025

Relax pinned dependencies, allowing adding parlant to existing projects

@kichanyurd
Copy link
Contributor

This is a good point, thank you.

Could you please specify which specific dependencies you ran into issues with?

@nurikk
Copy link
Author

nurikk commented Oct 5, 2025

This is a good point, thank you.

Could you please specify which specific dependencies you ran into issues with?

There were no particular issues with dependencies, just the issue with pinned versions. For example specific fast api version fastapi = "0.115.12", all can be seen here.

https://github.com/emcie-co/parlant/pull/581/files#diff-50c86b7ed8ac2cf95bd48334961bf0530cdc77b5a56f852c5c61b89d735fd711

I could use something like --force flag installing dependencies, but I'd rather not

@kichanyurd
Copy link
Contributor

Hey mate @nurikk could you please revert files changed just due to formatting?

Let's keep the PR focused on only the right files, and move forward with merging it 🙏

@nurikk nurikk force-pushed the relax_dependencies branch from 42bb49c to 4ffd39f Compare October 9, 2025 09:41
Signed-off-by: nurikk <1525421+nurikk@users.noreply.github.com>
@nurikk nurikk force-pushed the relax_dependencies branch from 4ffd39f to 5afdcd0 Compare October 9, 2025 09:56
@nurikk
Copy link
Author

nurikk commented Oct 9, 2025

Hey mate @nurikk could you please revert files changed just due to formatting?

Let's keep the PR focused on only the right files, and move forward with merging it 🙏

done!

@kichanyurd
Copy link
Contributor

@nurikk One last request: Can you please remove poetry.lock from the PR? It will always tend to create conflicts, yet I can just run poetry.lock myself in a separate commit, which is easier.

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

Labels

None yet

2 participants