[CI] Update doc translate script (vllm-project#11644)

update doc translate script to make it work as expect

- vLLM version: v0.23.0
- vLLM main:
vllm-project/vllm@1f486d9

Signed-off-by: wangxiyuan <wangxiyuan1007@gmail.com>

Author: wangxiyuan

.github/workflows/schedule_doc_translate.yaml

@@ -105,19 +105,20 @@ jobs:
         run: |
           python .github/workflows/scripts/po_translate.py \
             --files "${{ steps.detect.outputs.files }}" \
-            --output-json /tmp/translation_results.json
+            --output-json /tmp/translation_results.json \
+            --ignore-validation-error
 
       - name: Validate translation coverage
         if: steps.translate.outcome == 'success'
         id: validate
-        continue-on-error: true
         run: |
           python .github/workflows/scripts/po_translate.py \
             --files "${{ steps.detect.outputs.files }}" \
-            --validate-only
+            --validate-only \
+            --ignore-validation-error
 
       - name: Process translated files
-        if: steps.validate.outcome == 'success'
+        if: steps.translate.outcome == 'success'
         id: results
         run: |
           [ ! -f /tmp/translation_results.json ] && echo "No results" && exit 1
@@ -151,7 +152,7 @@ jobs:
           } >> $GITHUB_OUTPUT
 
       - name: Commit and push
-        if: steps.validate.outcome == 'success'
+        if: steps.translate.outcome == 'success'
         env:
           GITHUB_TOKEN: ${{ secrets.PAT_TOKEN }}
         run: |
@@ -161,7 +162,7 @@ jobs:
           git push -f fork "${{ env.BRANCH_NAME }}"
 
       - name: Create PR in upstream
-        if: steps.validate.outcome == 'success'
+        if: steps.translate.outcome == 'success'
         uses: actions/github-script@v9
         env:
           FILE_LIST: ${{ steps.results.outputs.file_list }}

.github/workflows/scripts/po_translate.py

@@ -303,11 +303,13 @@ def _clean_response(response: str) -> str:
         return response
 
 
-def validate_coverage(files_arg: str) -> int:
+def validate_coverage(files_arg: str, ignore_error: bool = False) -> int:
     """Check that every msgstr in the given PO files is non-empty.
 
     Prints a per-file summary and returns 0 when all files pass,
-    1 when any file has untranslated entries.
+    1 when any file has untranslated entries (unless *ignore_error* is True,
+    in which case untranslated entries only produce warnings and the exit
+    code is always 0).
     """
     file_list = [f.strip() for f in files_arg.split(",") if f.strip()]
     total_entries = 0
@@ -332,7 +334,8 @@ def validate_coverage(files_arg: str) -> int:
         untranslated += len(empty)
 
         if empty:
-            print(f"  FAIL: {path.name} — {len(empty)}/{file_entries} untranslated")
+            label = "WARN" if ignore_error else "FAIL"
+            print(f"  {label}: {path.name} — {len(empty)}/{file_entries} untranslated")
             for e in empty[:5]:
                 preview = e.msgid[:80].replace("\n", "\\n")
                 print(f'         msgid="{preview}..."')
@@ -346,6 +349,10 @@ def validate_coverage(files_arg: str) -> int:
         f"\nCoverage: {total_entries - untranslated}/{total_entries} translated "
         f"({untranslated} missing) in {len(file_list)} file(s)"
     )
+    if ignore_error:
+        if failed_files:
+            print("Validation errors ignored (--ignore-validation-error).")
+        return 0
     return 1 if failed_files else 0
 
 
@@ -360,10 +367,16 @@ async def async_main():
         action="store_true",
         help="Only validate translation coverage, do not translate",
     )
+    parser.add_argument(
+        "--ignore-validation-error",
+        action="store_true",
+        help="When used with --validate-only, print warnings instead of failing on untranslated entries. "
+        "When used without --validate-only, translation failures do not cause a non-zero exit code.",
+    )
     args = parser.parse_args()
 
     if args.validate_only:
-        return validate_coverage(args.files)
+        return validate_coverage(args.files, ignore_error=args.ignore_validation_error)
 
     api_key = args.api_key or os.getenv("DEEPSEEK_API_KEY")
     if not api_key:
@@ -391,6 +404,10 @@ async def async_main():
     out.write_text(json.dumps(results, indent=2, ensure_ascii=False), encoding="utf-8")
 
     print(f"\nResult: {len(success_files)}/{len(file_list)} translated -> {args.output_json}")
+    if args.ignore_validation_error:
+        if len(success_files) < len(file_list):
+            print("Translation errors ignored (--ignore-validation-error), continuing with successful files.")
+        return 0
     return 0 if success_files else 1