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

Adding pyautogen as optional dependency in pyproject.toml #613

Closed
wants to merge 11 commits into from
Closed

Adding pyautogen as optional dependency in pyproject.toml #613

wants to merge 11 commits into from

Conversation

hemanthsavasere
Copy link
Contributor

Please describe the purpose of this pull request.
Fixing the bug related while setup of the repository.

How to test
Setup a fresh new project in any IDE, autogen dependency was not met. After running "poetry add pyautogen --optional" command dependency was resolved.

Have you tested this PR?
Have you tested the latest commit on the PR? If so please provide outputs from your tests.
Yes. After running "poetry add pyautogen" dependency was resolved. Autogen dependency was not found in pyproject.toml file

Related issues or PRs
Please link any related GitHub issues or [PRs]
N.A

Is your PR over 500 lines of code?
If so, please break up your PR into multiple smaller PRs so that we can review them quickly, or provide justification for its length.
No

Additional context
Added

@hemanthsavasere
Copy link
Contributor Author

@cpacker can you please review this ?

@cpacker cpacker self-requested a review December 12, 2023 18:31
Copy link
Collaborator

@cpacker cpacker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you fix the formatting issues? Check the contributor guide for running black

@hemanthsavasere
Copy link
Contributor Author

Can you fix the formatting issues? Check the contributor guide for running black
Yes, ran the command poetry run black -l 140 . but it has introduced many changes in other files that my current PR is not associated with.

@cpacker
Copy link
Collaborator

cpacker commented Dec 14, 2023

Seems like the code formatter messed up the code diff - I made a quick PR and merged it in here ##616 and added you as a co-author to it.

@cpacker cpacker closed this Dec 14, 2023
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