jiayuasu commented on PR #2924: URL: https://github.com/apache/sedona/pull/2924#issuecomment-4403866540
All 10 Copilot comments flag pre-existing issues in the **English source** documents, not bugs in the translation. The Chinese versions in this PR faithfully preserve the English content verbatim (modulo translated prose), so each issue exists identically in both files. Verified against `master`: | Comment | Same issue in English source | |---|---| | `docker.zh.md:71` stray `-e` | `docs/setup/docker.md:71` ✅ | | `install-scala.zh.md:48,73` Spark 3.3 example | `docs/setup/install-scala.md:43,68` ✅ | | `install-python.zh.md:73,98` Spark 3.3 example | `docs/setup/install-python.md:71,97` ✅ | | `emr.zh.md:26` EMR 6.9.0 / Spark 3.3.0 | `docs/setup/emr.md:20` ✅ | | `fabric.zh.md:131` trailing comma in JSON | `docs/setup/fabric.md:131` ✅ | | `flink/install-python.zh.md:25` `attr` should be `attrs` | `docs/setup/flink/install-python.md:25` ✅ | | `cluster.zh.md:27` JRE 1.8 vs current Java 11+ requirement | `docs/setup/cluster.md:27` ✅ | | `azure-synapse-analytics.zh.md:83` malformed Maven coordinates | `docs/setup/azure-synapse-analytics.md:82-83` ✅ | The scope of this PR is translation only. Fixing these in the Chinese files alone would let the two locales drift apart — the next person regenerating the English source would re-introduce the discrepancy. They're better addressed in a separate follow-up PR that updates both EN and ZH copies together (and the parallel install/setup pages can be audited as a unit while we're at it). Happy to open that follow-up PR after this one merges if it's helpful. -- 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]
