pre-commit fix

Author: Girish5tri

.github/workflows/bug_triager_workflow.yml

@@ -31,4 +31,4 @@ jobs:
           MAINTAINERS: ${{ secrets.MAINTAINERS }}
           REPO_CONFIG: ${{ secrets.REPO_CONFIG }}
         run: |
-          python -m triager --bugs -c example-config.yaml --log --send-email
+          python -m triager --bugs -c config.yaml --log --send-email

.github/workflows/ci_workflow.yml

@@ -31,4 +31,4 @@ jobs:
           MAINTAINERS: ${{ secrets.MAINTAINERS }}
           REPO_CONFIG: ${{ secrets.REPO_CONFIG }}
         run: |
-          python -m triager --ci -c example-config.yaml --log --send-email
+          python -m triager --ci -c config.yaml --log --send-email

.gitignore

@@ -3,4 +3,5 @@
 *.code-workspace
 __pycache__/
 *.egg-info
-.env
+.env
+.flake8

LICENSE

@@ -671,4 +671,4 @@ into proprietary programs.  If your program is a subroutine library, you
 may consider it more useful to permit linking proprietary applications with
 the library.  If this is what you want to do, use the GNU Lesser General
 Public License instead of this License.  But first, please read
-<https://www.gnu.org/licenses/why-not-lgpl.html>.
+<https://www.gnu.org/licenses/why-not-lgpl.html>.

example-config.yaml

@@ -1,13 +1,13 @@
-##Template for config.yaml
+# #Template for config.yaml
 
-#Organization name (used in email subject)
-organization_name: "Ansible Networking"
+# #Organization name (used in email subject)
+# organization_name: "Ansible Networking"
 
-#Workflow name
-workflow_name: "tests.yml"
+# #Workflow name
+# workflow_name: "tests.yml"
 
-#Time delta (in days) for triaging issues
-timedelta: 14
+# #Time delta (in days) for triaging issues
+# timedelta: 14
 ##Template for adding repos in Github secrets or .env file(for running locally)
 
 # REPO_CONFIG= {

requirements.txt

@@ -1,4 +1,4 @@
 ptable
 PyYAML
 requests
-python-dotenv
+python-dotenv

triager/__main__.py

@@ -1,19 +1,23 @@
 import argparse
 import logging
 from datetime import datetime
-import os
+
 from dotenv import load_dotenv
 
 from triager import triager
+from triager.ci_report import generate_ci_report
 from triager.config import Config
 from triager.mailer import send_mail
 from triager.release import __ver__
 from triager.tablemaker import make_table
-from triager.ci_report import generate_ci_report
+
 
 def run(args):
     logging_level = logging.DEBUG if args.debug else logging.INFO
-    logging.basicConfig(level=logging_level, format='%(levelname)-10s%(message)s')
+    logging.basicConfig(
+        level=logging_level,
+        format="%(levelname)-10s%(message)s",
+    )
 
     load_dotenv()
 
@@ -27,30 +31,48 @@ def run(args):
                 table = make_table(issues)
                 print(table)
                 if args.send_email and config.is_email_ready:
-                    send_mail(content=table, config=config, subject=f"{config.organization_name} Weekly Triage - {report_date}")
+                    subject = (
+                        f"{config.organization_name} Weekly Triage - " f"{report_date}"
+                    )
+                    send_mail(content=table, config=config, subject=subject)
             else:
                 message = "No new issues found or error occurred during triage."
                 print(message)
                 if args.send_email and config.is_email_ready:
-                    send_mail(content=message, config=config, subject=f"{config.organization_name} Weekly Triage - No New Issues - {report_date}")
+                    subject = (
+                        f"{config.organization_name} Weekly Triage - "
+                        f"No New Issues - {report_date}"
+                    )
+                    send_mail(content=message, config=config, subject=subject)
         elif args.ci:
             ci_report = generate_ci_report(config)
             if ci_report:
                 table = make_table(ci_report, ci=True)
                 print(table)
-                report_date = ci_report.get("date", datetime.now().strftime("%Y-%m-%d"))
+                report_date = ci_report.get(
+                    "date",
+                    datetime.now().strftime("%Y-%m-%d"),
+                )
                 status = ci_report.get("overall_status", "Unknown")
                 if args.send_email and config.is_email_ready:
-                    send_mail(content=table, config=config, subject=f"{config.organization_name} Nightly CI Report - {report_date} - {status}")
+                    subject = (
+                        f"{config.organization_name} Nightly CI Report - "
+                        f"{report_date} - {status}"
+                    )
+                    send_mail(content=table, config=config, subject=subject)
             else:
-                logging.warning("No CI report generated or error occurred during CI report generation.")
+                logging.warning(
+                    "No CI report generated or error occurred during CI report "
+                    "generation.",
+                )
     except Exception as e:
         logging.error(f"An error occurred: {str(e)}", exc_info=True)
         raise
 
+
 def main():
     parser = argparse.ArgumentParser(
-        description="Triage issues and pull-requests from repositories of interest.",
+        description=("Triage issues and pull-requests from repositories of interest."),
         prog="Ansible Network Triager",
     )
     parser.add_argument(
@@ -65,32 +87,32 @@ def main():
     group.add_argument(
         "--bugs",
         action="store_true",
-        help="Generate a bug scrub report"
+        help="Generate a bug scrub report",
     )
     group.add_argument(
         "--ci",
         action="store_true",
-        help="Generate a CI report"
+        help="Generate a CI report",
     )
     parser.add_argument(
         "--log-to-file",
         nargs="?",
         const="/tmp/triager-{0}.log".format(
-            datetime.now().strftime("%Y-%m-%d-%X")
+            datetime.now().strftime("%Y-%m-%d-%X"),
         ),
         dest="log_to_file",
         help="save logging information to a file",
     )
     parser.add_argument(
         "--log",
         action="store_true",
-        help="display logging data on console"
+        help="display logging data on console",
     )
 
     parser.add_argument(
         "--debug",
         action="store_true",
-        help="Bump logging level to debug"
+        help="Bump logging level to debug",
     )
 
     parser.add_argument(
@@ -102,7 +124,7 @@ def main():
     parser.add_argument(
         "--version",
         action="store_true",
-        help="show version number"
+        help="show version number",
     )
 
     args = parser.parse_args()

triager/ci_report.py

@@ -1,23 +1,20 @@
 import logging
-import smtplib
 import textwrap
 from datetime import date
-from email.headerregistry import Address
-from email.message import EmailMessage
 
-import prettytable
 import requests
 
 WRAPPER = textwrap.TextWrapper(width=120)
 REQUEST_FMT = "https://api.github.com/repos/{0}/actions/workflows/{1}/runs"
 
+
 def generate_ci_report(config):
     data = []
     overall_status = "Success"
     workflow = config.workflow_name
-    
+
     for repo in config.ci_repos:
-        repo_name = repo['name'] if isinstance(repo, dict) else repo
+        repo_name = repo["name"] if isinstance(repo, dict) else repo
         try:
             resp = requests.get(
                 REQUEST_FMT.format(repo_name, workflow),
@@ -32,21 +29,30 @@ def generate_ci_report(config):
                 logging.error("API rate limiting or authentication issues.")
             logging.debug(f"Response content: {resp.content}")
             continue
-        
+
         resp_json = resp.json()
         if not resp_json["workflow_runs"]:
             logging.warning(f"No workflow runs found for {repo_name}")
             continue
-        
+
         latest_run = resp_json["workflow_runs"][0]
         status = latest_run["conclusion"]
+
+        if status is None:
+            logging.info(f"Workflow for {repo_name} is still in progress")
+            continue
+
         if status != "success":
             overall_status = "Failure"
-        
-        temp = {"repo": repo_name, "status": status, "url": latest_run["html_url"]}
+
+        temp = {
+            "repo": repo_name,
+            "status": status,
+            "url": latest_run["html_url"],
+        }
         logging.info(f"CI data fetched for {repo_name}: {status}")
         data.append(temp)
-        
+
     return {
         "data": data,
         "overall_status": overall_status,

triager/config.py

@@ -91,8 +91,7 @@ def _set_repo_config(self):
         logging.debug(f"Bug repos: {self.bug_repos}")
         logging.debug(f"CI repos: {self.ci_repos}")
 
-        # Parse maintainers
-
+    # Parse maintainers
     def _set_maintainers(self):
         maintainers_json = os.environ.get("MAINTAINERS", "[]")
         logging.debug(f"MAINTAINERS env var: {maintainers_json}")
@@ -108,8 +107,7 @@ def _set_maintainers(self):
 
         logging.debug(f"Maintainers: {self.maintainers}")
 
-        # Parse email configuration
-
+    # Parse email configuration
     def _set_email_config(self):
         self.sender = {
             "email": os.environ.get("EMAIL_SENDER"),
@@ -119,10 +117,11 @@ def _set_email_config(self):
         if self.sender["email"] and self.sender["password"]:
             logging.info("Email configuration found in environment variables")
         else:
-            logging.warning("Email configuration not found in environment variables")
-
-        # Set last triage date
+            logging.warning(
+                "Email configuration not found in environment variables",
+            )
 
+    # Set last triage date
     def _set_last_triage_date(self):
         self.last_triage_date = datetime.utcnow() - timedelta(
             days=int(self.config_data.get("timedelta", 14)),
@@ -147,5 +146,5 @@ def token(self):
     @property
     def is_email_ready(self):
         return bool(
-            self.sender["email"] and self.sender["password"] and self.maintainers
+            self.sender["email"] and self.sender["password"] and self.maintainers,
         )

triager/mailer.py

@@ -2,10 +2,12 @@
 import smtplib
 from datetime import date
 from email.message import EmailMessage
+
 from triager.ci_report import generate_ci_report
 
+
 def send_mail(content, config, subject):
-    logging.info("attempting to send email to maintainers") 
+    logging.info("attempting to send email to maintainers")
     msg = EmailMessage()
     msg["From"] = config.sender["email"]
     msg["To"] = [str(maintainer) for maintainer in config.maintainers]
@@ -14,11 +16,11 @@ def send_mail(content, config, subject):
     msg.set_content(str(content))
     msg.add_alternative(
         content.get_html_string(
-            attributes={"border": "1", "style": "text-align:center"}
+            attributes={"border": "1", "style": "text-align:center"},
         ),
         subtype="html",
     )
-    
+
     try:
         with smtplib.SMTP("smtp.gmail.com", 587) as smtp:
             logging.info("attempting to send email")
@@ -29,14 +31,19 @@ def send_mail(content, config, subject):
     except Exception as e:
         logging.error(f"Failed to send email: {str(e)}")
 
+
 def send_bug_report(content, config):
-    subject = f"{config.organization_name} Weekly Triage - Bug Report - {date.today().isoformat()}"
+    subject = (
+        f"{config.organization_name} Weekly Triage - "
+        f"Bug Report - {date.today().isoformat()}"
+    )
     send_mail(content, config, subject)
 
+
 def send_ci_report(content, config):
     ci_report = generate_ci_report(config)
     status = ci_report.get("overall_status", "Unknown")
     report_date = ci_report.get("date", date.today().isoformat())
-    
+
     subject = f"{config.organization_name} Nightly CI Report - {report_date} - {status}"
-    send_mail(content, config, subject)
+    send_mail(content, config, subject)