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

bug fix: remove unused get_pr_description method from Azure DevOps #1210

Merged
merged 1 commit into from
Sep 9, 2024

Conversation

mrT23
Copy link
Collaborator

@mrT23 mrT23 commented Sep 9, 2024

PR Type

Bug fix


Description

  • Removed the unused get_pr_description method from the Azure DevOps provider class
  • The method was responsible for retrieving the PR description and optionally clipping it to a maximum token length
  • This change simplifies the codebase by removing unnecessary functionality
  • No replacement or alternative method was added, suggesting the functionality was no longer needed

Changes walkthrough 📝

Relevant files
Code cleanup
azuredevops_provider.py
Remove unused PR description retrieval method                       

pr_agent/git_providers/azuredevops_provider.py

  • Removed the get_pr_description method from the Azure DevOps provider
    class
  • The removed method was responsible for retrieving and potentially
    clipping the PR description
  • +0/-6     

    💡 PR-Agent usage:
    Comment /help on the PR to get a list of all available PR-Agent tools and their descriptions

    Copy link
    Contributor

    PR Reviewer Guide 🔍

    ⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
    🏅 Score: 95
    🧪 No relevant tests
    🔒 No security concerns identified
    🔀 No multiple PR themes
    ⚡ No key issues to review

    Copy link
    Contributor

    PR Code Suggestions ✨

    No code suggestions found for the PR.

    @mrT23 mrT23 merged commit 6ed5537 into main Sep 9, 2024
    2 checks passed
    @mrT23 mrT23 deleted the tr/configurations branch September 9, 2024 13:26
    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.

    3 participants