Skip to content

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
    @github-actions
    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/"

    @github-actions
    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

    auto-merge was automatically disabled July 31, 2025 23:11

    Pull request was closed

    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.

    2 participants