From 1593d8932baf10e9fc044ce45b9b5a6032404ced Mon Sep 17 00:00:00 2001
From: mrT23 <tal.r@codium.ai>
Date: Fri, 15 Mar 2024 12:37:50 +0200
Subject: [PATCH] Refactor code in pr_code_suggestions.py and remove
 'use_repo_settings_file' from azure.md and automations_and_usage.md

---
 docs/docs/installation/azure.md                | 1 -
 docs/docs/usage-guide/automations_and_usage.md | 1 -
 pr_agent/tools/pr_code_suggestions.py          | 8 ++------
 3 files changed, 2 insertions(+), 8 deletions(-)

diff --git a/docs/docs/installation/azure.md b/docs/docs/installation/azure.md
index 8c111581d..e8c116642 100644
--- a/docs/docs/installation/azure.md
+++ b/docs/docs/installation/azure.md
@@ -4,7 +4,6 @@ To use Azure DevOps provider use the following settings in configuration.toml:
 ```
 [config]
 git_provider="azure"
-use_repo_settings_file=false
 ```
 
 Azure DevOps provider supports [PAT token](https://learn.microsoft.com/en-us/azure/devops/organizations/accounts/use-personal-access-tokens-to-authenticate?view=azure-devops&tabs=Windows) or [DefaultAzureCredential](https://learn.microsoft.com/en-us/azure/developer/python/sdk/authentication-overview#authentication-in-server-environments) authentication.
diff --git a/docs/docs/usage-guide/automations_and_usage.md b/docs/docs/usage-guide/automations_and_usage.md
index 81f182b54..b39f7e026 100644
--- a/docs/docs/usage-guide/automations_and_usage.md
+++ b/docs/docs/usage-guide/automations_and_usage.md
@@ -170,7 +170,6 @@ To use Azure DevOps provider use the following settings in configuration.toml:
 ```
 [config]
 git_provider="azure"
-use_repo_settings_file=false
 ```
 
 Azure DevOps provider supports [PAT token](https://learn.microsoft.com/en-us/azure/devops/organizations/accounts/use-personal-access-tokens-to-authenticate?view=azure-devops&tabs=Windows) or [DefaultAzureCredential](https://learn.microsoft.com/en-us/azure/developer/python/sdk/authentication-overview#authentication-in-server-environments) authentication.
diff --git a/pr_agent/tools/pr_code_suggestions.py b/pr_agent/tools/pr_code_suggestions.py
index 51dbe130c..d4f3556a4 100644
--- a/pr_agent/tools/pr_code_suggestions.py
+++ b/pr_agent/tools/pr_code_suggestions.py
@@ -393,11 +393,7 @@ def generate_summarized_suggestions(self, data: Dict) -> str:
 
             for label, suggestions in suggestions_labels.items():
                 num_suggestions=len(suggestions)
-                # pr_body += f"""<tr><td><strong>{label}</strong></td>"""
                 pr_body += f"""<tr><td rowspan={num_suggestions}><strong>{label.capitalize()}</strong></td>\n"""
-                # pr_body += f"""<td>"""
-                # pr_body += f"""<details><summary>{len(suggestions)} suggestions</summary>"""
-                # pr_body += f"""<table>"""
                 for i, suggestion in enumerate(suggestions):
 
                     relevant_file = suggestion['relevant_file'].strip()
@@ -444,11 +440,11 @@ def generate_summarized_suggestions(self, data: Dict) -> str:
 {example_code}                   
 """
                     pr_body += f"</details>"
-
                     pr_body += f"</td></tr>"
 
+
                 # pr_body += "</details>"
-                pr_body += """</td></tr>"""
+                # pr_body += """</td></tr>"""
             pr_body += """</tr></tbody></table>"""
             return pr_body
         except Exception as e: