You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@opendal.apache.org by "erickguan (via GitHub)" <gi...@apache.org> on 2026/06/20 14:17:59 UTC

[PR] Cache docusaurus build cache [opendal]

erickguan opened a new pull request, #7806:
URL: https://github.com/apache/opendal/pull/7806

   # Which issue does this PR close?
   
   <!--
   We generally require a GitHub issue to be filed for all bug fixes and enhancements. You can link an issue to this PR using the GitHub syntax. For example `Closes #123` indicates that this PR will close issue #123.
   -->
   
   Closes #.
   
   # Rationale for this change
   
   Improve website documentation generation speed.
   
   # What changes are included in this PR?
   
   - use `@docusaurus/faster` (introduced in 3.8). read [more](https://docusaurus.io/docs/api/docusaurus-config)
   - cache build artifacts
   - clean docs workflow
   
   # Are there any user-facing changes?
   
   
   <!--
   If there are user-facing changes then we may require documentation to be updated before approving the PR.
   -->
   
   <!---
   If there are any breaking changes to public APIs, please add the `breaking-changes` label.
   -->
   
   # AI Usage Statement
   
   None


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@opendal.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] ci: cache docusaurus build cache [opendal]

Posted by "Xuanwo (via GitHub)" <gi...@apache.org>.
Xuanwo merged PR #7806:
URL: https://github.com/apache/opendal/pull/7806


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@opendal.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] ci: cache docusaurus build cache [opendal]

Posted by "erickguan (via GitHub)" <gi...@apache.org>.
erickguan commented on code in PR #7806:
URL: https://github.com/apache/opendal/pull/7806#discussion_r3447140929


##########
.github/workflows/docs.yml:
##########
@@ -483,93 +486,63 @@ jobs:
           cache-dependency-path: "website/pnpm-lock.yaml"
 
       - name: Corepack
-        working-directory: website
-        run: npm i -g --force corepack && corepack enable
+        run: npm install -g --force corepack && corepack enable
 
-      - name: Download rust docs
+      - name: Download rust documentation
         uses: actions/download-artifact@v8
         with:
-          name: rust-docs
-          path: ./website/static/docs/rust
+          name: rust-documentation
+          path: website/static/docs/
 
-      - name: Download nodejs docs
+      - name: Download bindings and integrations documentations
         uses: actions/download-artifact@v8
         with:
-          name: nodejs-docs
-          path: ./website/static/docs/nodejs
+          pattern: *-documentation
+          path: website/static/docs/

Review Comment:
   Yes, fixed and rebased.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@opendal.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] ci: cache docusaurus build cache [opendal]

Posted by "tisonkun (via GitHub)" <gi...@apache.org>.
tisonkun commented on code in PR #7806:
URL: https://github.com/apache/opendal/pull/7806#discussion_r3446780924


##########
.github/workflows/docs.yml:
##########
@@ -483,93 +486,63 @@ jobs:
           cache-dependency-path: "website/pnpm-lock.yaml"
 
       - name: Corepack
-        working-directory: website
-        run: npm i -g --force corepack && corepack enable
+        run: npm install -g --force corepack && corepack enable
 
-      - name: Download rust docs
+      - name: Download rust documentation
         uses: actions/download-artifact@v8
         with:
-          name: rust-docs
-          path: ./website/static/docs/rust
+          name: rust-documentation
+          path: website/static/docs/
 
-      - name: Download nodejs docs
+      - name: Download bindings and integrations documentations
         uses: actions/download-artifact@v8
         with:
-          name: nodejs-docs
-          path: ./website/static/docs/nodejs
+          pattern: *-documentation
+          path: website/static/docs/

Review Comment:
   `rust-documentation` would be covered by this pattern again?



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@opendal.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org