Skip to content

refactor: remove unused config bind encoder param#418

Merged
banteg merged 1 commit intomasterfrom
fix/unused-config-bind-encoder-param
Mar 13, 2026
Merged

refactor: remove unused config bind encoder param#418
banteg merged 1 commit intomasterfrom
fix/unused-config-bind-encoder-param

Conversation

@banteg
Copy link
Copy Markdown
Owner

@banteg banteg commented Mar 13, 2026

Summary

  • remove the unused player_index parameter from _encode_player_bind_block
  • update the two keybind encoder callers to use the simplified signature
  • keep the already-verified config/input/UI behavior unchanged

Verification

  • uv run pytest tests/grim/test_grim_config.py tests/input/test_local_input.py tests/ui/test_controls_labels.py tests/ui/test_controls_layout.py
  • uv run ruff check src/grim/config.py tests/grim/test_grim_config.py
  • uv run ty check src/grim/config.py tests/grim/test_grim_config.py

@banteg banteg merged commit 2362997 into master Mar 13, 2026
8 of 9 checks passed
@banteg banteg deleted the fix/unused-config-bind-encoder-param branch March 13, 2026 19:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant