Skip to content

update pyproject urls fpr PyPI #53

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

misrasaurabh1
Copy link
Contributor

@misrasaurabh1 misrasaurabh1 commented Mar 14, 2025

PR Type

  • Enhancement

Description

  • Add new [project.urls] section for detailed project URLs

  • Remove redundant homepage entry from root level

  • Enhance metadata configuration for PyPI and documentation


Changes walkthrough 📝

Relevant files
Enhancement
pyproject.toml
Add detailed [project.urls] block in pyproject.toml           

pyproject.toml

  • Removed the single homepage configuration
  • Added [project.urls] block with four URL keys:
    * Homepage
    *
    Changelog
    * Repository
    * Documentation
  • +6/-1     

    Need help?
  • Type /help how to ... in the comments thread for any questions about PR-Agent usage.
  • Check out the documentation for more information.
  • @misrasaurabh1 misrasaurabh1 enabled auto-merge March 14, 2025 05:48
    Copy link

    PR Reviewer Guide 🔍

    Here are some key observations to aid the review process:

    ⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪
    🧪 No relevant tests
    🔒 No security concerns identified
    ⚡ Recommended focus areas for review

    Configuration Update

    Confirm that the new [project.urls] section aligns with metadata standards for PyPI and that the removal of the homepage field from the root level is intentional and correctly configured.

    [project.urls]
    Homepage = "https://codeflash.ai"
    Changelog = "https://github.com/codeflash-ai/codeflash/releases"
    Repository = "https://github.com/codeflash-ai/codeflash"
    Documentation = "https://docs.codeflash.ai/"

    Copy link

    PR Code Suggestions ✨

    No code suggestions found for the PR.

    @misrasaurabh1
    Copy link
    Contributor Author

    @KRRT7 can you make these changes alongside your uv migration? they way we define the urls is different for uv. This will also fix the errors here

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

    Successfully merging this pull request may close these issues.

    1 participant