beyond-all-reason / beyond-all-reason/RecoilEngine
Get rid of `cloakCost{,Moving}`
- Dominant language
- C++
- Stars
- 679
- Forks
- 290
- Avg merge
- 3d 2h
- Merged PRs (30d)
- 40
Description
It doesn't actually do anything (the engine does NOT use it for cloak costs), other than being re-exposed for reading back by Lua which is what customparams are for. Make PRs to games (incl. ancillaries like Circuit/Barbarian AI) first though since it should be trivial to do so.
Contributor guide
Research direction
Start by searching the engine and game integrations for `cloakCost` and `cloakCostMoving`, including the named Circuit and Barbarian AI ancillaries. First update the affected games and ancillaries as requested, then remove the unused engine exposure and verify that Lua customparams still provide the intended readback.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp
- Domain
- game-dev
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100