fix(ci): print cloudflare summary in comments

also update i18n docs so we trigger docs build.
maiieul · Jul 18, 2024 · e3c868c · e3c868c
1 parent 4f40c29
commit e3c868c
Show file tree

Hide file tree

Showing 2 changed files with 38 additions and 19 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -251,12 +251,9 @@ jobs:
     name: Bundle Qwik
     if: |
       always() &&
-      (needs.build-qwik.result == 'success' ||
-       needs.build-qwik.result == 'skipped') &&
-      (needs.build-linux-wasm-bindings.result == 'success' ||
-       needs.build-linux-wasm-bindings.result == 'skipped') &&
-      (needs.build-other-bindings.result == 'success' ||
-       needs.build-other-bindings.result == 'skipped')
+      (needs.build-qwik.result == 'success' || needs.build-qwik.result == 'skipped') &&
+      (needs.build-linux-wasm-bindings.result == 'success' || needs.build-linux-wasm-bindings.result == 'skipped') &&
+      (needs.build-other-bindings.result == 'success' || needs.build-other-bindings.result == 'skipped')
     runs-on: ubuntu-latest
     needs:
       - build-qwik
@@ -507,25 +504,28 @@ jobs:
             packages/docs/server
 
       - name: Cloudflare Pages Deployment
-        uses: cloudflare/pages-action@v1
+        # not the official version, so be careful when updating
+        uses: AdrianGonz97/refined-cf-pages-action@c7b767eef599f127d8ce2388d4ed2270d3b750af
         with:
           apiToken: ${{ secrets.CLOUDFLARE_API_TOKEN }}
           accountId: ${{ secrets.CLOUDFLARE_ACCOUNT_ID }}
           projectName: 'qwik-docs'
           directory: packages/docs/dist
-          gitHubToken: ${{ secrets.GITHUB_TOKEN }}
+          githubToken: ${{ secrets.GITHUB_TOKEN }}
 
   ############ UNIT TEST ############
   test-unit:
     name: Unit Tests
     if: |
       always() && (
-        ((needs.changes.outputs.build-qwik == 'true' ||
+        github.ref == 'refs/heads/main' || (
+          (
+            needs.changes.outputs.build-qwik == 'true' ||
             needs.changes.outputs.build-others == 'true' ||
-            needs.changes.outputs.build-rust == 'true') &&
+            needs.changes.outputs.build-rust == 'true'
+          ) &&
           needs.build-other-packages.result == 'success'
-        ) || 
-        github.ref == 'refs/heads/main'
+        )
       )
     runs-on: ubuntu-latest
     needs:
@@ -571,13 +571,14 @@ jobs:
     name: E2E Tests
     if: |
       always() && (
-        ((needs.changes.outputs.build-qwik == 'true' ||
-            needs.changes.outputs.build-others == 'true' ||
-            needs.changes.outputs.build-rust == 'true') &&
+        github.ref == 'refs/heads/main' || (
           needs.build-other-packages.result == 'success' &&
-          needs.test-unit.result == 'success'
-        ) || 
-        github.ref == 'refs/heads/main'
+          needs.test-unit.result == 'success' && (
+            needs.changes.outputs.build-qwik == 'true' ||
+            needs.changes.outputs.build-others == 'true' ||
+            needs.changes.outputs.build-rust == 'true'
+          )
+        )
       )
 
     needs:
@@ -833,7 +834,15 @@ jobs:
       - changes
       - release
 
-    if: always() && (needs.changes.outputs.build-qwik == 'true' || needs.changes.outputs.build-rust == 'true' || needs.changes.outputs.build-others == 'true') && needs.release.result == 'success' && github.ref_name == 'main'
+    if: |
+      always() &&
+      (
+        needs.changes.outputs.build-qwik == 'true' ||
+        needs.changes.outputs.build-rust == 'true' ||
+        needs.changes.outputs.build-others == 'true'
+      ) &&
+      needs.release.result == 'success' &&
+      github.ref_name == 'main'
 
     steps:
       - name: Repository Dispatch

diff --git a/packages/docs/src/routes/docs/integrations/i18n/index.mdx b/packages/docs/src/routes/docs/integrations/i18n/index.mdx
@@ -106,6 +106,16 @@ Alternatively, you can use the `deepl-localize` command to translate your string
 }
 ```
 
+### compiled-i18n
+
+[compiled-i18n](https://github.com/wmertens/compiled-i18n) is inspired by the $localize system from Angular. It only requires a plugin to be added to the Vite configuration.
+
+It supports both runtime and compile-time translations.
+
+See the [Qwik-specific instructions](https://github.com/wmertens/compiled-i18n/blob/main/qwik.md).
+
+Automatic translation is supported via [deepl-localize](https://github.com/tzdesign/deepl-localize).
+
 ### qwik-speak
 
 [qwik-speak](https://github.com/robisim74/qwik-speak) library to translate texts, dates and numbers in Qwik apps.