beyond-all-reason / beyond-all-reason/RecoilEngine

`Spring.SetAlly` being one-way is probably a bad default

Open
#1,657 0 comments 0 reactions 0 assignees View on GitHub
area: Lua API
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

Open the contributing 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.