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

Remove `groupclear` etc bindings

Open
#992 0 comments 0 reactions 0 assignees View on GitHub
good first issue
Dominant language
C++
Stars
679
Forks
290
Avg merge
3d 1h
Merged PRs (30d)
38

Description

grep `groupclear` in the whole engine. There's default bindings but these don't actually do anything, the actual bind is `group unset`. This only serves to confuse people (for example https://springrts.com/phpbb/viewtopic.php?f=1&t=44462 and https://github.com/beyond-all-reason/Beyond-All-Reason/pull/1983 ). Part of the task is to check for other obsolete `groupfoo` binds and remove/replace them as well.

Contributor guide

Open the contributing guide

Research direction

Start by grepping the whole engine for `groupclear`, then inspect the default bindings and compare them with the working `group unset` binding. Search for other obsolete `groupfoo` bindings, remove or replace them as appropriate, and consider the work done when no nonfunctional obsolete group bindings remain.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
game-dev
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.