Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Updates CONTRIBUTING.md to add installation of uv in development setup #1971

Merged
merged 3 commits into from
Feb 2, 2025

Conversation

Harsh-2909
Copy link
Contributor

@Harsh-2909 Harsh-2909 commented Feb 1, 2025

Description

  • Summary of changes: This PR adds a point in CONTRIBUTING.md file for installing uv when setting up a development environment.
  • Related issues: This PR is not connected to any issue.
  • Motivation and context: I just made a fork of this repo to understand and contribute to this project. When I was setting up development environment from the instructions, the setup was incomplete due to uv not being installed in my system.
  • Environment or dependencies: No changes there
  • Impact on metrics: NA.

Type of change

Please check the options that are relevant:

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Model update (Addition or modification of models)
  • Other (please describe): Documentation change

Checklist

  • Adherence to standards: Code complies with Agno’s style guidelines and best practices.
  • Formatting and validation: You have run ./scripts/format.sh and ./scripts/validate.sh to ensure code is formatted and linted.
  • Self-review completed: A thorough review has been performed by the contributor(s).
  • Documentation: Docstrings and comments have been added or updated for any complex logic.
  • Examples and guides: Relevant cookbook examples have been included or updated (if applicable).
  • Tested in a clean environment: Changes have been tested in a clean environment to confirm expected behavior.
  • Tests (optional): Tests have been added or updated to cover any new or changed functionality.

@Harsh-2909 Harsh-2909 requested a review from a team as a code owner February 1, 2025 09:13
@dirkbrnd
Copy link
Contributor

dirkbrnd commented Feb 2, 2025

Thanks for the addition!

@dirkbrnd dirkbrnd merged commit a1affc1 into agno-agi:main Feb 2, 2025
@Harsh-2909 Harsh-2909 deleted the docs/contributing branch February 3, 2025 18:32
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants