Adds Irish Sea map#4546
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
✅ Files skipped from review due to trivial changes (1)
🚧 Files skipped from review as they are similar to previous changes (1)
WalkthroughAdds a new "Irish Sea" map: registers ChangesIrish Sea map addition
Estimated code review effort: 2 (Simple) | ~10 minutes Possibly related PRs
Suggested labels: Suggested reviewers: Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@map-generator/assets/maps/irishsea/info.json`:
- Around line 311-314: The map entry for the Llangollen location has a typo in
its name field; update the JSON object in the Irish Sea map data so the `name`
value for this coordinate entry uses the correct spelling "Llangollen" instead
of "Llangollenes". Make the change in the source data entry so the generated
manifest picks it up automatically.
In `@resources/maps/irishsea/manifest.json`:
- Around line 209-212: The map entry name is misspelled as “Llangollenes” and
should be corrected to “Llangollen” in both the Irish Sea manifest and the
corresponding source info.json record. Update the affected “name” field in the
map data so the town label is consistent across both files.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 83889382-078a-483e-98c0-3527aee73abc
⛔ Files ignored due to path filters (10)
map-generator/assets/maps/irishsea/image.pngis excluded by!**/*.pngresources/flags/GB-CHE.svgis excluded by!**/*.svgresources/flags/GB-CMA.svgis excluded by!**/*.svgresources/flags/GB-LAN.svgis excluded by!**/*.svgresources/flags/GB-Merseyside.svgis excluded by!**/*.svgresources/flags/Ireland Green Harp.svgis excluded by!**/*.svgresources/flags/Kingdom of Ireland.svgis excluded by!**/*.svgresources/maps/irishsea/map.binis excluded by!**/*.binresources/maps/irishsea/map16x.binis excluded by!**/*.binresources/maps/irishsea/map4x.binis excluded by!**/*.bin
📒 Files selected for processing (6)
map-generator/assets/maps/irishsea/info.jsonresources/countries.jsonresources/lang/en.jsonresources/maps/irishsea/manifest.jsonresources/maps/irishsea/thumbnail.webpsrc/core/game/Maps.gen.ts
Description:
Adds map of the Irish Sea: https://en.wikipedia.org/wiki/Irish_Sea
A relatively small-medium size map of 970k tiles, centered around Isle of Man (slightly modified to be more centered), surrounded in all sides by land.
Since all trade will go around the island, this will create crazy strong pirate players, which have become popular thanks to Youtubers and the Tradeship buff in v32.
19 default nations, with 43 extra nations for a total of 62, for the Humans vs Nations gamemode and Solo games.
irish_sea.mp4
Please complete the following:
Please put your Discord username so you can be contacted if a bug or regression is found:
tri.star1011