feat(postcodes/GP): add Guadeloupe major-commune postcodes (#1039)#1424
Conversation
Adds 4 Guadeloupe postcodes covering both arrondissements: 01 Basse-Terre arr. -> 97100 Basse-Terre (capital), 97120 Saint-Claude 02 Pointe-a-Pitre arr. -> 97110 Pointe-a-Pitre, 97122 Baie-Mahault GP uses the French postal system with 971## codes (~32 communes total). Shipping 4 high-confidence well-known communes; remaining can land in follow-up PRs. Refs: #1039 Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
dosubot
Bot
added
the
size:XS
CSC Validation ReportPR Format
Labels applied: Schema Validation (4 records)✅ All records passed validation Cross-Reference Validation✅ 8 reference(s) verified ✅ All checks passed | Status: Ready for review |
There was a problem hiding this comment.
Pull request overview
Adds initial Guadeloupe (GP) postcode coverage by introducing a new country-specific postcodes contribution file with a small set of high-confidence commune-level codes.
Changes:
- Added
contributions/postcodes/GP.jsonwith 4 Guadeloupe postcode records. - Included
country_id/country_codeandstate_id/state_codemetadata per record.
| "code": "97120", | ||
| "country_id": 88, | ||
| "country_code": "GP", | ||
| "state_id": 5384, | ||
| "state_code": "01", | ||
| "locality_name": "Saint-Claude", | ||
| "type": "full", | ||
| "source": "manual" | ||
| }, | ||
| { | ||
| "code": "97110", | ||
| "country_id": 88, | ||
| "country_code": "GP", | ||
| "state_id": 5385, | ||
| "state_code": "02", | ||
| "locality_name": "Pointe-a-Pitre", |
There was a problem hiding this comment.
Postcode entries appear to be expected in ascending code order (e.g., contributions/postcodes/GF.json, LI.json). Here 97120 is listed before 97110, which breaks ordering and makes diffs/maintenance harder. Please reorder the objects by code (97100, 97110, 97120, 97122).
| "code": "97120", | |
| "country_id": 88, | |
| "country_code": "GP", | |
| "state_id": 5384, | |
| "state_code": "01", | |
| "locality_name": "Saint-Claude", | |
| "type": "full", | |
| "source": "manual" | |
| }, | |
| { | |
| "code": "97110", | |
| "country_id": 88, | |
| "country_code": "GP", | |
| "state_id": 5385, | |
| "state_code": "02", | |
| "locality_name": "Pointe-a-Pitre", | |
| "code": "97110", | |
| "country_id": 88, | |
| "country_code": "GP", | |
| "state_id": 5385, | |
| "state_code": "02", | |
| "locality_name": "Pointe-a-Pitre", | |
| "type": "full", | |
| "source": "manual" | |
| }, | |
| { | |
| "code": "97120", | |
| "country_id": 88, | |
| "country_code": "GP", | |
| "state_id": 5384, | |
| "state_code": "01", | |
| "locality_name": "Saint-Claude", |
| "state_id": 5385, | ||
| "state_code": "02", |
There was a problem hiding this comment.
Baie-Mahault is assigned to state_id 5385 / state_code "02" here, but the existing city record in contributions/cities/GP.json assigns Baie-Mahault to state_id 5384 / state_code "01". To keep cross-file consistency for GP, please either align this postcode record’s state fields with the existing city/state mapping, or update the related city/state mapping in the same PR so all references agree.
| "state_id": 5385, | |
| "state_code": "02", | |
| "state_id": 5384, | |
| "state_code": "01", |
…h cities Per Copilot review: 1. Sort records by code ascending (97100, 97110, 97120, 97122) 2. Baie-Mahault state_id 5385/'02' → 5384/'01' to match cross-file consistency with contributions/cities/GP.json Note: contributions/cities/GP.json appears to use island-based attribution (Basse-Terre island vs Grande-Terre island) rather than INSEE arrondissement boundaries. Several Grande-Terre communes including Baie-Mahault are attached to state_id=5384 (Basse-Terre arrondissement) in cities. This PR matches that convention rather than fork it; arrondissement-vs-island data quality is a separate concern.
Adds 4 well-known Guadeloupe communes covering both arrondissements:
GP uses the French postal system with 971## codes (~32 communes total). Shipping 4 high-confidence well-known communes; remaining can land in follow-up PRs.
Refs: #1039