bitflicker64 commented on PR #557: URL: https://github.com/apache/hugegraph-doc/pull/557#issuecomment-6559038492
## Review summary: R2 (FULL) **Reviewed:** head `01b8113f` against base `8f74072b` (master after the #499 merge), all 50 files. Previous round: [R1 summary](https://github.com/apache/hugegraph-doc/pull/557#issuecomment-6556233018), at head `7db47aaa`. **Verdict:** no blocking issues found. Every R1 finding is fixed; the status of each is posted in its original thread. Three Minor items remain. **Score: 9.3 / 10.** This is the mean of 2 of 4 reviewers so far, so it is provisional. | Reviewer | Raw | Adjusted | Deductions | |---|---|---|---| | Reviewer A (model not reported) | 9.5 | 9.5 | 1 Minor | | Reviewer B (Claude Opus 5.5) | 9.0 | 9.0 | 2 Minor | | Reviewer C | pending | | | | Reviewer D | pending | | | **Minor:** - [Version notes removed from the `restful-api` index, auth and graphs pages](https://github.com/apache/hugegraph-doc/pull/557#discussion_r4435960373) - [The "rather than a JSON object" note on raw-text POST](https://github.com/apache/hugegraph-doc/pull/557#discussion_r4435960377) - [Old deep-section anchors not kept on the merged Java page](https://github.com/apache/hugegraph-doc/pull/557#discussion_r4435960379) **R1 correction:** the R1 finding about the Cypher `Authorization` header was wrong. Master has allowed anonymous Cypher queries when auth is disabled since apache/hugegraph#3366; details are in [the R1 thread](https://github.com/apache/hugegraph-doc/pull/557#discussion_r4435962031). The current text is accurate. The R1 summary is left unchanged as the historical record of that round. **Checks:** `dist/validate-links.sh` passes, and a strict Hugo 0.168.0 build passes. Navigation built from the latest, 1.7, 1.5, 1.3 and 1.0 snapshots matches the expected stats. No full historical artifact build or browser check was done. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
