Merge pull request #201 from balancer/fix_contract_list_gen

fix: Update package version and handle chains with no deployments.
balancer · May 24, 2024 · 4824e44 · 4824e44
2 parents 638630a + 509684a
commit 4824e44
Show file tree

Hide file tree

Showing 6 changed files with 22 additions and 273 deletions.
diff --git a/.github/python_actions/genContractLists.py b/.github/python_actions/genContractLists.py
@@ -1,6 +1,7 @@
 import requests
 import pandas as pd
 import re
+import os
 from bal_addresses import AddrBook, GITHUB_DEPLOYMENTS_NICE
 
 OUTPUT_PATH = "docs/reference/contracts/deployment-addresses"
@@ -19,8 +20,13 @@
 SCANNERS_BY_CHAIN = AddrBook.chains["SCANNERS_BY_CHAIN"]
 
 def address_directory(chain, status=None):
-    r = requests.get(f"https://raw.githubusercontent.com/balancer/balancer-deployments/master/addresses/{chain}.json")
-    r=r.json()
+    try:
+        r = requests.get(f"https://raw.githubusercontent.com/balancer/balancer-deployments/master/addresses/{chain}.json")
+        r.raise_for_status()
+        r=r.json()
+    except:
+        print (f"Error fetching deployments {chain} addresses, returning empty dict dict")
+        return {}
     if isinstance(status, str):
         return {k: v for k, v in r.items() if v['status'] == status}
     else:
@@ -51,7 +57,6 @@ def genFullTable(r, chain):
 
 def genPoolFactories(r, chain):
     result = pd.DataFrame(columns=["Contract", "Address", "Deployment"])
-    print(f"Generating pools for {chain}")
     for deployment, depdata in r.items():
         if "-pool" not in deployment:
             continue
@@ -126,6 +131,7 @@ def genFromContractList(r, chain, contractList):
     return result
 
 def genChainMd(chain):
+    print(f"Generating md for {chain}")
     groupedContracts = []
     for contracts in CONTRACTS_BY_HEADING.values():
         groupedContracts += contracts
@@ -194,9 +200,16 @@ def genChainMd(chain):
 
 def main():
     for chain in SCANNERS_BY_CHAIN:
-        output=genChainMd(chain)
-        with open(f"{OUTPUT_PATH}/{chain}.md", "w") as f:
-            f.write(output)
+        if AddrBook(chain).deployments:
+            output=genChainMd(chain)
+            with open(f"{OUTPUT_PATH}/{chain}.md", "w") as f:
+                f.write(output)
+        else:
+            #Remove old files if they exist
+            try:
+                os.remove(f"{OUTPUT_PATH}/{chain}.md")
+            except:
+                pass
 
 if __name__ == "__main__":
    main()
diff --git a/.github/python_actions/requirements.txt b/.github/python_actions/requirements.txt
@@ -1,4 +1,4 @@
-git+https://github.com/BalancerMaxis/bal_addresses@0.8.4
+git+https://github.com/BalancerMaxis/bal_addresses@0.9.4
 pandas>=1.5.3
 tabulate>=0.9.0
 
diff --git a/.github/workflows/update_contract_addresses.yaml b/.github/workflows/update_contract_addresses.yaml
@@ -17,7 +17,7 @@ jobs:
     - name: Install Python
       uses: actions/setup-python@v4
       with:
-        python-version: '3.9'
+        python-version: '3.10'
 
     - name: Update Files
       id: update

diff --git a/.github/workflows/update_permissions_table.yaml b/.github/workflows/update_permissions_table.yaml
@@ -17,7 +17,7 @@ jobs:
     - name: Install Python
       uses: actions/setup-python@v4
       with:
-        python-version: '3.9'
+        python-version: '3.10'
 
     - name: Update Files
       id: update

diff --git a/docs/reference/contracts/deployment-addresses/fantom.md b/docs/reference/contracts/deployment-addresses/fantom.md