🐛 fix: fix error when ~/.config/dns-manager does not exist (#3)

* 🐛 fix: fix error when `~/.config/dns-manager` does not exist * 🐛 fix: fix action
zrr-lab · Nov 4, 2024 · 955fea8 · 955fea8
1 parent 60f5a5d
commit 955fea8
Show file tree

Hide file tree

Showing 3 changed files with 29 additions and 23 deletions.
diff --git a/.github/workflows/release.yml b/.github/workflows/release.yml
@@ -6,9 +6,6 @@ on:
 jobs:
   build:
     runs-on: ubuntu-latest
-    strategy:
-      matrix:
-        target-version: [py310, py311, py312]
     concurrency: release
     permissions:
       id-token: write
@@ -28,9 +25,10 @@ jobs:
     - name: Install the latest version of uv 📦
       uses: astral-sh/setup-uv@v2
     - name: Build distributions 📦
-      env:
-        PYFUTURE_TARGET: ${{ matrix.target-version }}
-      run: uv build
+      run: |
+        PYFUTURE_TARGET=py312 uv build --wheel
+        PYFUTURE_TARGET=py311 uv build --wheel
+        PYFUTURE_TARGET=py310 uv build --wheel
     - name: Publish package distributions to PyPI 🚀
       if: steps.release.outputs.released == 'true'
       uses: pypa/gh-action-pypi-publish@release/v1

diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -10,20 +10,25 @@ permissions:
 
 jobs:
   coverage:
-    runs-on: ubuntu-22.04
+    runs-on: ubuntu-latest
     steps:
-      - name: Checkout 📦
-        uses: actions/checkout@v4
+      - uses: actions/checkout@v4
+      - name: Install just 📦
+        uses: extractions/setup-just@v2
       - name: Install the latest version of uv 📦
         uses: astral-sh/setup-uv@v3
       - name: Install Python 📦
         uses: actions/setup-python@v4
         with:
           python-version: "3.12"
-      - name: Install just 📦
-        uses: extractions/setup-just@v2
+      - name: Install dependencies 📦
+        run: just install
+      - run: |
+          uv run dnsm update examples/simple.toml
         env:
-          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+          TENCENTCLOUD_SECRET_ID: ${{ secrets.TENCENTCLOUD_SECRET_ID }}
+          TENCENTCLOUD_SECRET_KEY: ${{ secrets.TENCENTCLOUD_SECRET_KEY }}
+          LEXICON_CLOUDFLARE_AUTH_TOKEN: ${{ secrets.LEXICON_CLOUDFLARE_AUTH_TOKEN }}
       - name: Run tests and benchmarks ✅
         uses: CodSpeedHQ/action@v3
         env:
@@ -33,9 +38,9 @@ jobs:
         with:
           run: just test
       - name: Upload coverage reports to Codecov 🚀
-        uses: codecov/codecov-action@v4-beta
+        uses: codecov/codecov-action@v4
         env:
           CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }}
         with:
-          flags: Pyfuture
+          flags: dns-manager
           plugins: pycoverage,compress-pycoverage
diff --git a/dns_manager/setter/base.py b/dns_manager/setter/base.py
@@ -106,15 +106,18 @@ def update_dns(self, remove_unmanaged: bool = False):
                     records.append((subdomain, record.value))
                     logger.warning(f"([red]🔓 Unmanaged[/]) {record}")
                 path = Path("~/.config/dns-manager/unmanaged.json")
-                save_config(
-                    path,
-                    Config(
-                        domain=self.domain,
-                        setter_name=self.setter_name,
-                        records=records,
-                    ).model_dump(),
-                )
-                logger.info(f"Unmanaged records saved to [bold purple]{path}[/].")
+                try:
+                    save_config(
+                        path,
+                        Config(
+                            domain=self.domain,
+                            setter_name=self.setter_name,
+                            records=records,
+                        ).model_dump(),
+                    )
+                    logger.info(f"Unmanaged records saved to [bold purple]{path}[/].")
+                except Exception as e:
+                    logger.warning(f"Failed to save unmanaged records: {e}")
         for subdomain, record in new_records.items():
             cached_record = self.cached_records.get(subdomain, None)
             if cached_record is None: