Ticket #42211

S3_0 alien Grassland has inconsistent mining_result & mining_time

Eröffnet am: 2021-05-10 21:36 Letztes Update: 2021-05-16 11:52

Auswertung:
Verantwortlicher:
Typ:
Status:
Geschlossen
Komponente:
Meilenstein:
Priorität:
5 - Mittel
Schweregrad:
5 - Mittel
Lösung:
Gefixt
Datei:
2

Details

S3_1 CI ruleup test for alien is failing, because S3_0 alien to upgrade from has mining_result = "no", but mining_time = 10 for Grassland. S3_1 now has a sanity check against that.

That's not hard error in S3_0 (the mining_time gets ignored when mining_result is "no"), and S3_0 is in semi-freeze before beta2 release. Still, maybe we consider fixing CI important enough to push in the fix now?

Ticket-Verlauf (3/7 Historien)

2021-05-10 21:36 Aktualisiert von: cazfi
  • New Ticket "S3_0 alien Grassland has inconsistent mining_result & mining_time" created
2021-05-10 21:45 Aktualisiert von: cazfi
  • Lösung Update from Keine to Accepted
  • Meilenstein Update from (Keine) to 2.6.5 (closed)
2021-05-10 21:48 Aktualisiert von: cazfi
Kommentar

Maybe S3_1 should convert _time to zero in this kind of situations?

2021-05-10 22:06 Aktualisiert von: cazfi
  • Komponente Update from Rulesets to General
Kommentar

Now one patch for S3_0/S2_6, to fix the ruleset, and another for S3_1, to convert from inconsistent S3_0 ruleset, attached. Going to push S3_1 one immediately as a CI fix. S3_0/S2_6 patch will wait until after 3.0.0-beta2 is out.

2021-05-16 11:52 Aktualisiert von: cazfi
  • Status Update from Offen to Geschlossen
  • Verantwortlicher Update from (Keine) to cazfi
  • Lösung Update from Accepted to Gefixt

Bearbeiten

You are not logged in. I you are not logged in, your comment will be treated as an anonymous post. » Anmelden