Address actionable PR #535 review comments
- fastkml/styles.py: Pair's class docstring and StyleMap.highlight's
docstring still only mentioned StyleUrl/Style; both now note the nested
StyleMap support added by the earlier bug fix (CodeRabbit).
- document_test.py::test_document_places: fix a docstring that had the
cause of the pinned diff backwards. The diff is real and the assertion is
correct (verified live), but it's caused by this test's explicit
precision=2 forcing "0.00" output against a fixture that already reads a
bare "0" (normalized earlier by scripts/resave_kml_fixtures.py) -- not by
fastkml "stripping trailing zeros" as the old docstring claimed.
- styles_test.py::test_style_map_error: assert the exact diff action list
instead of len(diff) == 5, matching the style used elsewhere in this
package (CodeRabbit).
Reviewed every other bot comment on the PR (CodeRabbit's Document-places.kml
"assert diff == []" suggestion is incorrect and not applied; CodeScene/
CodeRabbit's shared-test-helper suggestions conflict with this suite's
deliberate explicit-test design; the resave script's .kml-only glob and the
Polygon winding-order nitpick were already-considered choices; Greptile/
Copilot's "docs/Document-clean.kml doesn't exist" is false and already
flagged isOutdated by GitHub; LlamaPReview's gx:Track concern was already
fixed in 00f4b98).
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
