beyond-all-reason / beyond-all-reason/RecoilEngine
`Spring.SetAlly` being one-way is probably a bad default
- Dominant language
- C++
- Stars
- 679
- Forks
- 290
- Avg merge
- 3d 2h
- Merged PRs (30d)
- 40
Description
Bill of materials:
* probably best to wait until #1656 gets resolved somehow, to avoid introducing new interface that will just get renamed again soon
----
See title, one-way allying is rarely desirable so the salient default should probably be mutual allying. Perhaps there should be a new `SetOneWayAlly` (or as appropriate if there was a rename due to #1656).
Contributor guide
Research direction
Start by reviewing the Spring.SetAlly interface and the unresolved dependency on #1656. Determine the final interface naming there, then define mutual allying as the default while preserving an explicitly one-way option; the issue is done when that API behavior and naming are settled and implemented.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp
- Domain
- game-dev
- Issue type
- Feature
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 25/100