Fix get_user_description in case pr_description.enable_pr_type=false

[zmeir]

Fixes an issue when getting the user description after a PR-Agent description was already generated, in case the configuration setting pr_description.enable_pr_type was false.

[coditamar]

/generate_labels

[barnett-yuxiang]

/generate_labels

[barnett-yuxiang]

No Unit Test code submitted?

[mrT23]

PR Analysis

(review updated until commit 7d89b82)

• 🎯 Main theme: This PR is about fixing an issue in the get_user_description function when the configuration setting pr_description.enable_pr_type is set to false.
• 📝 PR summary: The PR fixes a bug in the get_user_description function of the git_provider.py file. The bug was causing incorrect behavior when the configuration setting pr_description.enable_pr_type was set to false. The fix involves changing the condition that checks the start of the description.
• 📌 Type of PR: Bug fix
• 🧪 Relevant tests added: No
• ⏱️ Estimated effort to review [1-5]: 1, because the PR is small and the change is straightforward.
• 🔒 Security concerns: No security concerns found

PR Feedback

• 💡 General suggestions: The PR is concise and addresses a specific issue which is good. However, it would be beneficial to add tests that cover this specific case to prevent future regressions.

• 🤖 Code feedback:

  • relevant file: pr_agent/git_providers/git_provider.py
    suggestion: Consider using a constant to store the headers ("## PR Type", "## PR Description") to avoid hardcoding them in multiple places. This will make the code more maintainable. [medium]
    relevant line: if not any(description.startswith(header) for header in ("## PR Type", "## PR Description")):

[mrT23]

Persistent review updated to latest commit 7d89b82

[mrT23]

Preparing review...

[mrT23]

Preparing review...

[mrT23]

Preparing review...