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

[In Progress]Add poetry to build and manage package #31

Conversation

pranjaldoshi96
Copy link

@pranjaldoshi96 pranjaldoshi96 commented May 30, 2023

This PR aim is to use poetry instead of setup.py to build and manage package. As most of the packages are now moving towards poetry.

@pranjaldoshi96
Copy link
Author

@drazvan Should I proceed with this?

@drazvan
Copy link
Collaborator

drazvan commented May 30, 2023

Thanks for starting this @pranjaldoshi96! Yes, after the 0.2.0 release (today/tomorrow), we'll migrate to poetry as well. It was on my TODO list.

On a different note, as per our Contribution policy, all commits must be signed (https://github.com/NVIDIA/NeMo-Guardrails/blob/main/CONTRIBUTING.md#signing-your-work). Make sure your commit(s) are signed and force push the branch again.

Thanks!

@drazvan
Copy link
Collaborator

drazvan commented Feb 7, 2024

In #240 we made the migration to pyproject.toml. We don't need to migrate to Poetry just yet. Closing this.

@drazvan drazvan closed this Feb 7, 2024
@drazvan drazvan added duplicate This issue or pull request already exists status: wontfix This will not be worked on labels Feb 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate This issue or pull request already exists status: wontfix This will not be worked on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants