mention PR url in body of staging PR

EESSI · Jan 2, 2024 · 50b09e7 · 50b09e7
1 parent 83227dc
commit 50b09e7
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/scripts/automated_ingestion/automated_ingestion.cfg.example b/scripts/automated_ingestion/automated_ingestion.cfg.example
@@ -42,7 +42,7 @@ failed_tarball_overview_issue_body = An error occurred while trying to get the c
     ```
     {error}
     ```
-pr_body = A new tarball has been staged.
+pr_body = A new tarball has been staged for {pr_url}.
     Please review the contents of this tarball carefully.
     Merging this PR will lead to automatic ingestion of the tarball.
 

diff --git a/scripts/automated_ingestion/eessitarball.py b/scripts/automated_ingestion/eessitarball.py
@@ -280,10 +280,14 @@ def make_approval_request(self):
         metadata = ''
         with open(self.local_metadata_path, 'r') as meta:
             metadata = meta.read()
+        meta_dict = json.loads(metadata)
+        repo, pr_id = meta_dict['link2pr']['repo'], meta_dict['link2pr']['pr']
+        pr_url = f"https://github.com/{repo}/pull/{pr_id}"
         # Try to get the tarball contents and open a PR to get approval for the ingestion
         try:
             tarball_contents = self.get_contents_overview()
             pr_body = self.config['github']['pr_body'].format(
+                pr_url=pr_url,
                 tar_overview=self.get_contents_overview(),
                 metadata=metadata,
             )