Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Polish existing update rules #15090

Merged
merged 2 commits into from May 2, 2018

Conversation

Projects
None yet
2 participants
@pchote
Copy link
Member

pchote commented Apr 28, 2018

This removes explicit line numbers from the reports because these are not going to be accurate when multiple rules are run together. It also tweaks some of the wording and moves the string addition operator to the end of the lines. I have tried to make sure that the rule descriptions explain both what has changed and what the rule does.

pchote added some commits Apr 28, 2018

Remove explicit line numbers from update rule reports.
Additions/removals by other rules in the set will almost certainly
invalidate these.  Reporting the block and file is the best we can
reasonably do.
@pchote

This comment has been minimized.

Copy link
Member Author

pchote commented Apr 28, 2018

Example output:

paul@Ragnarok:OpenRA+remove-update-linenumbers *$ mono --debug OpenRA.Utility.exe cnc2 --update-mod release-20180307 --detailed
Found 9 API changes:
  * RemoveTerrainTypeIsWaterFlag: Remove TerrainType IsWater flag
     The IsWater flag on terrain type definitions has been unused for some time.
     This flag has now been removed from the tileset yaml.

  * RemoveWeaponScanRadius: Remove Weapon ScanRadius parameters
     The *ScanRadius parameters have been removed from weapon projectiles and warheads.
     These values are now automatically determined by the engine.
     CreateEffect.ImpactActors: False has been added to replace VictimScanRadius: 0

  * SplitTurretAimAnimation: Introduce WithTurretAimAnimation trait
     WithSpriteTurret.AimSequence and WithTurretAttackAnimation.AimSequence
     have been split into a new WithTurretAimAnimation trait.

  * DefineSoundDefaults: Move mod-specific sound defaults to yaml
     Mod-specific default sound values have been removed from several traits
     (ParaDrop.ChuteSound, EjectOnDeath.ChuteSound, ProductionParadrop.ChuteSound, Building.BuildSounds, Building.UndeploySounds)
     Uses of these traits are listed for inspection so the values can be overriden in yaml.

  * RenameWormSpawner: WormSpawner renamed and generalized to ActorSpawner
     The D2k-specific WormSpawner trait was renamed to ActorSpawner, generalized,
     and moved into the common mod code. Uses of the old traits are updated to account for this.

  * RemoveWithReloadingSpriteTurret: Remove WithReloadingSpriteTurret trait
     WithReloadingSpriteTurret has been superseded by conditions.
     Instances of this trait are replaced by WithSpriteTurret.

  * IgnoreAbstractActors: Abstract actors are ignored while parsing rules
     Actor ids starting with '^' are now reserved for abstract inheritance templates.
     Definitions that may be affected are listed for inspection so that they can be renamed if necessary.

  * AddEditorPlayer: Add EditorPlayer
     Map editor now requires an EditorPlayer to avoid loading unnecessary player traits.
     A warning is displayed if this actor has not been defined.

  * RemovePaletteFromCurrentTileset: Remove PaletteFromCurrentTileset trait
     The PaletteFromCurrentTileset trait and Palette field on TileSets have been removed.
     Terrain palettes are now explicitly defined on the world actor.
     Palette definitions are generated based on the Tileset metadata.

Run this command with the --apply flag to apply the update rules.

paul@Ragnarok:OpenRA+remove-update-linenumbers *$ mono --debug OpenRA.Utility.exe cnc2 --update-mod release-20180307 --detailed --apply
WARNING: This command will automatically rewrite your mod rules.
Side effects of this command may include changing the whitespace to 
match the default conventions, and any yaml comments will be removed.

We strongly recommend that you have a backup of your mod rules, and 
for best results, to use a Git client to review the line-by-line 
changes and discard any unwanted side effects.

Press y to continue, or any other key to cancel: y

RemoveTerrainTypeIsWaterFlag: Remove TerrainType IsWater flag
   Updating mod... COMPLETE
   Updating system maps... COMPLETE

RemoveWeaponScanRadius: Remove Weapon ScanRadius parameters
   Updating mod... COMPLETE
   Updating system maps... COMPLETE

SplitTurretAimAnimation: Introduce WithTurretAimAnimation trait
   Updating mod... COMPLETE
   Updating system maps... COMPLETE

DefineSoundDefaults: Move mod-specific sound defaults to yaml
   Updating mod... COMPLETE
   Updating system maps... COMPLETE
   Manual changes are required to complete this update:
    * The default value for Building.BuildSounds has been removed.
      You may wish to explicitly define `BuildSounds: placbldg.aud, build5.aud` on the `Building` trait 
      definitions on the following actors (if they have not already been inherited from a parent).
       * ^Building (rules/defaults.yaml)
       * ^CivBuildingHusk (rules/defaults.yaml)
       * ^CivFieldHusk (rules/defaults.yaml)
       * ^Tree (rules/defaults.yaml)
       * ^TreeHusk (rules/defaults.yaml)
       * ^TibTree (rules/defaults.yaml)
       * ^Rock (rules/defaults.yaml)
       * FACT (rules/structures.yaml)
       * NUKE (rules/structures.yaml)
       * NUK2 (rules/structures.yaml)
       * PROC (rules/structures.yaml)
       * SILO (rules/structures.yaml)
       * PYLE (rules/structures.yaml)
       * HAND (rules/structures.yaml)
       * AFLD (rules/structures.yaml)
       * WEAP (rules/structures.yaml)
       * HPAD (rules/structures.yaml)
       * HQ (rules/structures.yaml)
       * FIX (rules/structures.yaml)
       * EYE (rules/structures.yaml)
       * TMPL (rules/structures.yaml)
       * GUN (rules/structures.yaml)
       * SAM (rules/structures.yaml)
       * GTWR (rules/structures.yaml)
       * ROCK1 (rules/trees.yaml)
       * ROCK2 (rules/trees.yaml)
       * ROCK4 (rules/trees.yaml)
       * ROCK5 (rules/trees.yaml)
       * ROCK6 (rules/trees.yaml)
       * ROCK7 (rules/trees.yaml)
       * T08 (rules/trees.yaml)
       * T08.Husk (rules/trees.yaml)
       * T09 (rules/trees.yaml)
       * T09.Husk (rules/trees.yaml)
       * T10 (rules/trees.yaml)
       * T10.Husk (rules/trees.yaml)
       * T11 (rules/trees.yaml)
       * T11.Husk (rules/trees.yaml)
       * T12.Husk (rules/trees.yaml)
       * T13.Husk (rules/trees.yaml)
       * T15 (rules/trees.yaml)
       * T15.Husk (rules/trees.yaml)
       * TC01 (rules/trees.yaml)
       * TC01.Husk (rules/trees.yaml)
       * TC02 (rules/trees.yaml)
       * TC02.Husk (rules/trees.yaml)
       * TC03 (rules/trees.yaml)
       * TC03.Husk (rules/trees.yaml)
       * TC04 (rules/trees.yaml)
       * TC04.Husk (rules/trees.yaml)
       * TC05 (rules/trees.yaml)
       * TC05.Husk (rules/trees.yaml)
       * V01 (rules/civilian.yaml)
       * V01.Husk (rules/civilian.yaml)
       * V02 (rules/civilian.yaml)
       * V02.Husk (rules/civilian.yaml)
       * V03 (rules/civilian.yaml)
       * V03.Husk (rules/civilian.yaml)
       * V04 (rules/civilian.yaml)
       * V04.Husk (rules/civilian.yaml)
       * V05 (rules/civilian.yaml)
       * V05.Husk (rules/civilian.yaml)
       * V06 (rules/civilian.yaml)
       * V06.Husk (rules/civilian.yaml)
       * V07 (rules/civilian.yaml)
       * V07.Husk (rules/civilian.yaml)
       * ARCO (rules/civilian.yaml)
       * ARCO.Husk (rules/civilian.yaml)
       * BRIDGE1 (rules/civilian.yaml)
       * BRIDGE2 (rules/civilian.yaml)
       * BRIDGE3 (rules/civilian.yaml)
       * BRIDGE4 (rules/civilian.yaml)
       * BRIDGEHUT (rules/civilian.yaml)
       * V20 (rules/civilian-desert.yaml)
       * V20.Husk (rules/civilian-desert.yaml)
       * V21 (rules/civilian-desert.yaml)
       * V21.Husk (rules/civilian-desert.yaml)
       * V22 (rules/civilian-desert.yaml)
       * V22.Husk (rules/civilian-desert.yaml)
       * V24 (rules/civilian-desert.yaml)
       * V24.Husk (rules/civilian-desert.yaml)
       * V25 (rules/civilian-desert.yaml)
       * V25.Husk (rules/civilian-desert.yaml)
       * V26 (rules/civilian-desert.yaml)
       * V26.Husk (rules/civilian-desert.yaml)
       * V30 (rules/civilian-desert.yaml)
       * V30.Husk (rules/civilian-desert.yaml)
       * V31 (rules/civilian-desert.yaml)
       * V31.Husk (rules/civilian-desert.yaml)
       * V32 (rules/civilian-desert.yaml)
       * V32.Husk (rules/civilian-desert.yaml)
       * V33 (rules/civilian-desert.yaml)
       * V33.Husk (rules/civilian-desert.yaml)
       * V37 (rules/civilian-desert.yaml)
       * V37.Husk (rules/civilian-desert.yaml)
       * V19 (rules/tech.yaml)
       * V19.Husk (rules/tech.yaml)
       * HOSP (rules/tech.yaml)
       * HOSP.Husk (rules/tech.yaml)
       * BIO (rules/tech.yaml)
       * BIO.Husk (rules/tech.yaml)
       * MISS (rules/tech.yaml)
    * The default value for Building.UndeploySounds has been removed.
      You may wish to explicitly define `UndeploySounds: cashturn.aud` on the `Building` trait 
      definitions on the following actors (if they have not already been inherited from a parent).
       * ^Building (rules/defaults.yaml)
       * ^BaseBuilding (rules/defaults.yaml)
       * ^CivBuildingHusk (rules/defaults.yaml)
       * ^CivFieldHusk (rules/defaults.yaml)
       * ^Wall (rules/defaults.yaml)
       * ^Tree (rules/defaults.yaml)
       * ^TreeHusk (rules/defaults.yaml)
       * ^TibTree (rules/defaults.yaml)
       * ^Rock (rules/defaults.yaml)
       * FACT (rules/structures.yaml)
       * NUKE (rules/structures.yaml)
       * NUK2 (rules/structures.yaml)
       * PROC (rules/structures.yaml)
       * SILO (rules/structures.yaml)
       * PYLE (rules/structures.yaml)
       * HAND (rules/structures.yaml)
       * AFLD (rules/structures.yaml)
       * WEAP (rules/structures.yaml)
       * HPAD (rules/structures.yaml)
       * HQ (rules/structures.yaml)
       * FIX (rules/structures.yaml)
       * EYE (rules/structures.yaml)
       * TMPL (rules/structures.yaml)
       * GUN (rules/structures.yaml)
       * SAM (rules/structures.yaml)
       * GTWR (rules/structures.yaml)
       * ROCK1 (rules/trees.yaml)
       * ROCK2 (rules/trees.yaml)
       * ROCK4 (rules/trees.yaml)
       * ROCK5 (rules/trees.yaml)
       * ROCK6 (rules/trees.yaml)
       * ROCK7 (rules/trees.yaml)
       * T08 (rules/trees.yaml)
       * T08.Husk (rules/trees.yaml)
       * T09 (rules/trees.yaml)
       * T09.Husk (rules/trees.yaml)
       * T10 (rules/trees.yaml)
       * T10.Husk (rules/trees.yaml)
       * T11 (rules/trees.yaml)
       * T11.Husk (rules/trees.yaml)
       * T12.Husk (rules/trees.yaml)
       * T13.Husk (rules/trees.yaml)
       * T15 (rules/trees.yaml)
       * T15.Husk (rules/trees.yaml)
       * TC01 (rules/trees.yaml)
       * TC01.Husk (rules/trees.yaml)
       * TC02 (rules/trees.yaml)
       * TC02.Husk (rules/trees.yaml)
       * TC03 (rules/trees.yaml)
       * TC03.Husk (rules/trees.yaml)
       * TC04 (rules/trees.yaml)
       * TC04.Husk (rules/trees.yaml)
       * TC05 (rules/trees.yaml)
       * TC05.Husk (rules/trees.yaml)
       * V01 (rules/civilian.yaml)
       * V01.Husk (rules/civilian.yaml)
       * V02 (rules/civilian.yaml)
       * V02.Husk (rules/civilian.yaml)
       * V03 (rules/civilian.yaml)
       * V03.Husk (rules/civilian.yaml)
       * V04 (rules/civilian.yaml)
       * V04.Husk (rules/civilian.yaml)
       * V05 (rules/civilian.yaml)
       * V05.Husk (rules/civilian.yaml)
       * V06 (rules/civilian.yaml)
       * V06.Husk (rules/civilian.yaml)
       * V07 (rules/civilian.yaml)
       * V07.Husk (rules/civilian.yaml)
       * ARCO (rules/civilian.yaml)
       * ARCO.Husk (rules/civilian.yaml)
       * BRIDGE1 (rules/civilian.yaml)
       * BRIDGE2 (rules/civilian.yaml)
       * BRIDGE3 (rules/civilian.yaml)
       * BRIDGE4 (rules/civilian.yaml)
       * BRIDGEHUT (rules/civilian.yaml)
       * V20 (rules/civilian-desert.yaml)
       * V20.Husk (rules/civilian-desert.yaml)
       * V21 (rules/civilian-desert.yaml)
       * V21.Husk (rules/civilian-desert.yaml)
       * V22 (rules/civilian-desert.yaml)
       * V22.Husk (rules/civilian-desert.yaml)
       * V24 (rules/civilian-desert.yaml)
       * V24.Husk (rules/civilian-desert.yaml)
       * V25 (rules/civilian-desert.yaml)
       * V25.Husk (rules/civilian-desert.yaml)
       * V26 (rules/civilian-desert.yaml)
       * V26.Husk (rules/civilian-desert.yaml)
       * V30 (rules/civilian-desert.yaml)
       * V30.Husk (rules/civilian-desert.yaml)
       * V31 (rules/civilian-desert.yaml)
       * V31.Husk (rules/civilian-desert.yaml)
       * V32 (rules/civilian-desert.yaml)
       * V32.Husk (rules/civilian-desert.yaml)
       * V33 (rules/civilian-desert.yaml)
       * V33.Husk (rules/civilian-desert.yaml)
       * V37 (rules/civilian-desert.yaml)
       * V37.Husk (rules/civilian-desert.yaml)
       * V19 (rules/tech.yaml)
       * V19.Husk (rules/tech.yaml)
       * HOSP (rules/tech.yaml)
       * HOSP.Husk (rules/tech.yaml)
       * BIO (rules/tech.yaml)
       * BIO.Husk (rules/tech.yaml)
       * MISS (rules/tech.yaml)
    * Map: /Users/paul/src/OpenRA/mods/cnc2/maps/gdi03:
       * The default value for Building.BuildSounds has been removed.
         You may wish to explicitly define `BuildSounds: placbldg.aud, build5.aud` on the `Building` trait 
         definitions on the following actors (if they have not already been inherited from a parent).
          * SAM (rules.yaml)
       * The default value for Building.UndeploySounds has been removed.
         You may wish to explicitly define `UndeploySounds: cashturn.aud` on the `Building` trait 
         definitions on the following actors (if they have not already been inherited from a parent).
          * SAM (rules.yaml)
    * Map: /Users/paul/src/OpenRA/mods/cnc2/maps/nod07c:
       * The default value for Building.BuildSounds has been removed.
         You may wish to explicitly define `BuildSounds: placbldg.aud, build5.aud` on the `Building` trait 
         definitions on the following actors (if they have not already been inherited from a parent).
          * HPAD.IN (rules.yaml)
       * The default value for Building.UndeploySounds has been removed.
         You may wish to explicitly define `UndeploySounds: cashturn.aud` on the `Building` trait 
         definitions on the following actors (if they have not already been inherited from a parent).
          * HPAD.IN (rules.yaml)

RenameWormSpawner: WormSpawner renamed and generalized to ActorSpawner
   Updating mod... COMPLETE
   Updating system maps... COMPLETE

RemoveWithReloadingSpriteTurret: Remove WithReloadingSpriteTurret trait
   Updating mod... COMPLETE
   Updating system maps... COMPLETE
   Manual changes are required to complete this update:
    * WithReloadingSpriteTurret has been replaced by WithSpriteTurret
      You should use AmmoPool.AmmoConditions to switch turret type when reloading
      to restore the previous behaviour on the following actors:
         * MLRS (rules/vehicles.yaml)

IgnoreAbstractActors: Abstract actors are ignored while parsing rules
   Updating mod... COMPLETE
   Updating system maps... COMPLETE
   Manual changes are required to complete this update:
    * Actor ids starting with '^' are now reserved for abstract
      inheritance templates, and will not be parsed by the game.
      Check the following definitions and rename them if they are not used for inheritance:
       * ^fact.colorpicker (rules/misc.yaml)
       * ^BaseWorld (rules/world.yaml)
       * ^Palettes (rules/palettes.yaml, rules/campaign-palettes.yaml)
       * ^ExistsInWorld (rules/defaults.yaml)
       * ^SpriteActor (rules/defaults.yaml)
       * ^1x1Shape (rules/defaults.yaml)
       * ^2x1Shape (rules/defaults.yaml)
       * ^2x2Shape (rules/defaults.yaml)
       * ^3x2Shape (rules/defaults.yaml)
       * ^GainsExperience (rules/defaults.yaml)
       * ^AutoTargetGround (rules/defaults.yaml)
       * ^AutoTargetGroundAssaultMove (rules/defaults.yaml)
       * ^AutoTargetAir (rules/defaults.yaml)
       * ^AutoTargetAll (rules/defaults.yaml)
       * ^AutoTargetAllAssaultMove (rules/defaults.yaml)
       * ^AcceptsCloakCrate (rules/defaults.yaml)
       * ^Vehicle (rules/defaults.yaml, rules/campaign-tooltips.yaml, rules/campaign-palettes.yaml)
       * ^Tank (rules/defaults.yaml, rules/campaign-tooltips.yaml, rules/campaign-palettes.yaml)
       * ^Helicopter (rules/defaults.yaml, rules/campaign-tooltips.yaml, rules/campaign-palettes.yaml)
       * ^Infantry (rules/defaults.yaml, rules/campaign-tooltips.yaml, rules/campaign-palettes.yaml)
       * ^Soldier (rules/defaults.yaml)
       * ^CivInfantry (rules/defaults.yaml)
       * ^ArmedCivilian (rules/defaults.yaml)
       * ^DINO (rules/defaults.yaml, rules/campaign-tooltips.yaml)
       * ^Viceroid (rules/defaults.yaml)
       * ^Plane (rules/defaults.yaml, rules/campaign-tooltips.yaml)
       * ^Ship (rules/defaults.yaml, rules/campaign-tooltips.yaml)
       * ^Building (rules/defaults.yaml, rules/campaign-tooltips.yaml)
       * ^BaseBuilding (rules/defaults.yaml)
       * ^CivBuilding (rules/defaults.yaml, rules/campaign-tooltips.yaml)
       * ^CivBuildingHusk (rules/defaults.yaml, rules/campaign-tooltips.yaml)
       * ^TechBuilding (rules/defaults.yaml, rules/campaign-tooltips.yaml)
       * ^CivField (rules/defaults.yaml)
       * ^CivFieldHusk (rules/defaults.yaml)
       * ^Wall (rules/defaults.yaml, rules/campaign-tooltips.yaml)
       * ^Tree (rules/defaults.yaml)
       * ^TreeHusk (rules/defaults.yaml)
       * ^TibTree (rules/defaults.yaml)
       * ^Rock (rules/defaults.yaml)
       * ^CommonHuskDefaults (rules/defaults.yaml, rules/campaign-tooltips.yaml, rules/campaign-palettes.yaml)
       * ^Husk (rules/defaults.yaml)
       * ^HelicopterHusk (rules/defaults.yaml)
       * ^Bridge (rules/defaults.yaml)
       * ^Crate (rules/defaults.yaml)
       * ^Defense (rules/defaults.yaml)
       * ^DisabledOverlay (rules/defaults.yaml)
    * Map: /Users/paul/src/OpenRA/mods/cnc2/maps/funpark01:
       * Actor ids starting with '^' are now reserved for abstract
         inheritance templates, and will not be parsed by the game.
         Check the following definitions and rename them if they are not used for inheritance:
          * ^Palettes (rules.yaml)
          * ^CivInfantry (rules.yaml)
          * ^CivBuilding (rules.yaml)
          * ^DINO (rules.yaml)
    * Map: /Users/paul/src/OpenRA/mods/cnc2/maps/gdi04c:
       * Actor ids starting with '^' are now reserved for abstract
         inheritance templates, and will not be parsed by the game.
         Check the following definitions and rename them if they are not used for inheritance:
          * ^Palettes (rules.yaml)
          * ^CivInfantry (rules.yaml)
          * ^Bridge (rules.yaml)
    * Map: /Users/paul/src/OpenRA/mods/cnc2/maps/gdi05a:
       * Actor ids starting with '^' are now reserved for abstract
         inheritance templates, and will not be parsed by the game.
         Check the following definitions and rename them if they are not used for inheritance:
          * ^Palettes (rules.yaml)
          * ^Building (rules.yaml)
    * Map: /Users/paul/src/OpenRA/mods/cnc2/maps/gdi05b:
       * Actor ids starting with '^' are now reserved for abstract
         inheritance templates, and will not be parsed by the game.
         Check the following definitions and rename them if they are not used for inheritance:
          * ^Palettes (rules.yaml)
          * ^Building (rules.yaml)
    * Map: /Users/paul/src/OpenRA/mods/cnc2/maps/gdi06:
       * Actor ids starting with '^' are now reserved for abstract
         inheritance templates, and will not be parsed by the game.
         Check the following definitions and rename them if they are not used for inheritance:
          * ^Bridge (rules.yaml)
    * Map: /Users/paul/src/OpenRA/mods/cnc2/maps/nod01:
       * Actor ids starting with '^' are now reserved for abstract
         inheritance templates, and will not be parsed by the game.
         Check the following definitions and rename them if they are not used for inheritance:
          * ^Palettes (rules.yaml)
          * ^Bridge (rules.yaml)
          * ^CivBuilding (rules.yaml)
          * ^CivInfantry (rules.yaml)
    * Map: /Users/paul/src/OpenRA/mods/cnc2/maps/nod02a:
       * Actor ids starting with '^' are now reserved for abstract
         inheritance templates, and will not be parsed by the game.
         Check the following definitions and rename them if they are not used for inheritance:
          * ^Palettes (rules.yaml)
          * ^Vehicle (rules.yaml)
          * ^Tank (rules.yaml)
          * ^Helicopter (rules.yaml)
          * ^Infantry (rules.yaml)
          * ^Plane (rules.yaml)
          * ^Ship (rules.yaml)
          * ^Building (rules.yaml)
          * ^Wall (rules.yaml)
          * ^CommonHuskDefaults (rules.yaml)
    * Map: /Users/paul/src/OpenRA/mods/cnc2/maps/nod04a:
       * Actor ids starting with '^' are now reserved for abstract
         inheritance templates, and will not be parsed by the game.
         Check the following definitions and rename them if they are not used for inheritance:
          * ^Palettes (rules.yaml)
          * ^Infantry (rules.yaml)
          * ^CivBuilding (rules.yaml)
          * ^CivBuildingHusk (rules.yaml)
    * Map: /Users/paul/src/OpenRA/mods/cnc2/maps/nod04b:
       * Actor ids starting with '^' are now reserved for abstract
         inheritance templates, and will not be parsed by the game.
         Check the following definitions and rename them if they are not used for inheritance:
          * ^Vehicle (rules.yaml)
          * ^Tank (rules.yaml)
    * Map: /Users/paul/src/OpenRA/mods/cnc2/maps/nod05:
       * Actor ids starting with '^' are now reserved for abstract
         inheritance templates, and will not be parsed by the game.
         Check the following definitions and rename them if they are not used for inheritance:
          * ^Palettes (rules.yaml)
    * Map: /Users/paul/src/OpenRA/mods/cnc2/maps/nod06a:
       * Actor ids starting with '^' are now reserved for abstract
         inheritance templates, and will not be parsed by the game.
         Check the following definitions and rename them if they are not used for inheritance:
          * ^Infantry (rules.yaml)
    * Map: /Users/paul/src/OpenRA/mods/cnc2/maps/nod06b:
       * Actor ids starting with '^' are now reserved for abstract
         inheritance templates, and will not be parsed by the game.
         Check the following definitions and rename them if they are not used for inheritance:
          * ^Palettes (rules.yaml)
    * Map: /Users/paul/src/OpenRA/mods/cnc2/maps/nod06c:
       * Actor ids starting with '^' are now reserved for abstract
         inheritance templates, and will not be parsed by the game.
         Check the following definitions and rename them if they are not used for inheritance:
          * ^Bridge (rules.yaml)
    * Map: /Users/paul/src/OpenRA/mods/cnc2/maps/nod07a:
       * Actor ids starting with '^' are now reserved for abstract
         inheritance templates, and will not be parsed by the game.
         Check the following definitions and rename them if they are not used for inheritance:
          * ^Bridge (rules.yaml)
          * ^CivBuilding (rules.yaml)
    * Map: /Users/paul/src/OpenRA/mods/cnc2/maps/nod07b:
       * Actor ids starting with '^' are now reserved for abstract
         inheritance templates, and will not be parsed by the game.
         Check the following definitions and rename them if they are not used for inheritance:
          * ^Bridge (rules.yaml)
    * Map: /Users/paul/src/OpenRA/mods/cnc2/maps/nod07c:
       * Actor ids starting with '^' are now reserved for abstract
         inheritance templates, and will not be parsed by the game.
         Check the following definitions and rename them if they are not used for inheritance:
          * ^CivBuilding (rules.yaml)
          * ^CivInfantry (rules.yaml)
          * ^BaseBuilding (rules.yaml)
    * Map: /Users/paul/src/OpenRA/mods/cnc2/maps/nod09:
       * Actor ids starting with '^' are now reserved for abstract
         inheritance templates, and will not be parsed by the game.
         Check the following definitions and rename them if they are not used for inheritance:
          * ^Bridge (rules.yaml)

AddEditorPlayer: Add EditorPlayer
   Updating mod... COMPLETE
   Updating system maps... COMPLETE
   Manual changes are required to complete this update:
    * The map editor now requires an EditorPlayer actor.
      Please add an EditorPlayer with the traits AlwaysVisible and Shroud to player.yaml
      (or a different rules yaml file of your choice).

RemovePaletteFromCurrentTileset: Remove PaletteFromCurrentTileset trait
   Updating mod... COMPLETE
   Updating system maps... COMPLETE
   Manual changes are required to complete this update:
    * You must add the following to your palette definitions:
      PaletteFromFile@terrain-desert:
          Name: terrain
          Tileset: DESERT
          Filename: desert.pal
          ShadowIndex: 4
      PaletteFromFile@terrain-winter:
          Name: terrain
          Tileset: WINTER
          Filename: winter.pal
          ShadowIndex: 4
      PaletteFromFile@terrain-snow:
          Name: terrain
          Tileset: SNOW
          Filename: snow.pal
          ShadowIndex: 4
      PaletteFromFile@terrain-temperat:
          Name: terrain
          Tileset: TEMPERAT
          Filename: temperat.pal
          ShadowIndex: 4
      PaletteFromFile@terrain-jungle:
          Name: terrain
          Tileset: JUNGLE
          Filename: jungle.pal
          ShadowIndex: 4
      PaletteFromFile@staticterrain-desert:
          Name: staticterrain
          Tileset: DESERT
          Filename: desert.pal
          ShadowIndex: 4
      PaletteFromFile@staticterrain-winter:
          Name: staticterrain
          Tileset: WINTER
          Filename: winter.pal
          ShadowIndex: 4
      PaletteFromFile@staticterrain-snow:
          Name: staticterrain
          Tileset: SNOW
          Filename: snow.pal
          ShadowIndex: 4
      PaletteFromFile@staticterrain-temperat:
          Name: staticterrain
          Tileset: TEMPERAT
          Filename: temperat.pal
          ShadowIndex: 4
      PaletteFromFile@staticterrain-jungle:
          Name: staticterrain
          Tileset: JUNGLE
          Filename: jungle.pal
          ShadowIndex: 4

Semi-automated update complete.
Please review the messages above for any manual actions that must be applied.

@pchote pchote referenced this pull request Apr 28, 2018

Merged

Add more update rules #15010

@abcdefg30 abcdefg30 merged commit efc744d into OpenRA:bleed May 2, 2018

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@abcdefg30

This comment has been minimized.

Copy link
Member

abcdefg30 commented May 2, 2018

@pchote pchote deleted the pchote:remove-update-linenumbers branch Jul 9, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.