terrymanu commented on PR #40137: URL: https://github.com/apache/shardingsphere/pull/40137#issuecomment-6095983751
### Result **Review Result: Mergeable** The local configuration changes match the three authorized values. No blocking issues were found. ### Evidence - `.github/workflows/nightly-check.yml:58`: Increased the dead-link job timeout to 30 minutes. - `.github/workflows/resources/linkcheck/markdown-link-check.json:47`: Reduced the retry limit to 3 and the fallback retry delay to 60 seconds. - Independent checks of baseline hashes, complete file differences, JSON/YAML parsing, and parameter placement passed. `git diff --check` exited with code 0. - Spotless checked the exact workflow file and exited with code 0, confirming coverage of one file. Checkstyle and RAT were not applicable; the JSON file is not covered by the configured formatter. - `Retry-After` may still override the fallback delay. These changes do not prove that the original CI run encountered HTTP 429, nor guarantee that all external link checks will complete. ### Coverage Standalone local candidate based on the pre-edit baseline `70a17f43be57c82ed8c61854083e336f1d0b7cd1`. The review covered both files and all three authorized line changes. Existing untracked files were preserved. The review followed the user-approved time budget for dead-link checking and introduced no new project commitments. Behavior, defensive code, impact, and verification checks were completed, with no new findings or unresolved evidence gaps. Inventory and ledger validation passed. This conclusion covers the local changes only; remote CI was neither run nor queried. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
