Skip to content

Update terrains.json - #1155

Closed
Xolgrim wants to merge 1 commit into
eressea:developfrom
Xolgrim:add-land-flagg-to-firewall
Closed

Update terrains.json#1155
Xolgrim wants to merge 1 commit into
eressea:developfrom
Xolgrim:add-land-flagg-to-firewall

Conversation

@Xolgrim

@Xolgrim Xolgrim commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Converted Feuerwand into a land region to prevent nearby ships from drifting away.

Feuerwand zu Landregion gemacht, damit Schiffe in der Nähe nicht mehr abtreiben können.
@ennorehling

Copy link
Copy Markdown
Member

Das geht so nicht, "land"hat eine Menge andere Nebenwirkungen. Mit dieser Änderung würdest du es z.B. unmöglich machen, das alte Datenfile zu laden :-)

Ohne einen Bugreport sehe ich keinen Grund, hier etwas zu ändern. Mit einem Bugreport könnte man einen Test schreiben. Dann könnte man gleich auch gucken, waum die Probleme dieser Änderung keinen Alarm ausgelöst haben (einen Test, der das Datenfile liest, haben wir wohl nicht im CI Build?).

@ennorehling
ennorehling self-requested a review August 24, 2026 06:17
@ennorehling

Copy link
Copy Markdown
Member

NB: Es ging hier um https://bugs.eressea.de/view.php?id=3119

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants