Summary
This PR significantly improves the coverage for Egyptian developers by expanding the egypt preset in presets.go.
Pull Request:
#107
Changes
- Corrected Typos: Fixed
tanda → tanta
- Comprehensive Coverage: Added all 27 Egyptian Governorates and their capitals
- Major Cities & Hubs: Included industrial/residential areas like:
- "6th of October", "10th of Ramadan", "New Cairo", "Obour City"
- "El Gouna", "Ain Shams", "Ain El Sokhna"
- Spelling Variations: Added common English transliterations for better matching:
asyut / asiutfayoum / faiyumsharkia / sharqiaal+mansurah / mansoura
- Colloquial Terms: Added
masr, om+el+donia, heliopolis, nasr+city
Why This Matters
GitHub's location field is free-text, so developers spell their cities differently. By adding variations, we maximize the chance of correctly identifying Egyptian developers regardless of how they spelled their location.
Testing
- Verified Go syntax is valid
- Confirmed URL encoding with
+ for spaces follows existing patterns in the codebase
Summary
This PR significantly improves the coverage for Egyptian developers by expanding the
egyptpreset in presets.go.Pull Request:
#107
Changes
tanda→tantaasyut/asiutfayoum/faiyumsharkia/sharqiaal+mansurah/mansouramasr,om+el+donia,heliopolis,nasr+cityWhy This Matters
GitHub's location field is free-text, so developers spell their cities differently. By adding variations, we maximize the chance of correctly identifying Egyptian developers regardless of how they spelled their location.
Testing
+for spaces follows existing patterns in the codebase