LiveSplit / LiveSplit/livesplit-core
Feature request: settings_list_set, settings_list_remove
Open
Nobody has claimed this yet.
- Dominant language
- Rust
- Stars
- 227
- Forks
- 81
- Avg merge
- 22h 45m
- Merged PRs (30d)
- 3
Description
Seems reasonable for the set and remove functions to return the replaced/removed value.
The set function would be equivalent to a combination of remove and the existing insert, but I still think it's worth putting set as its own function.
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Locate the settings list implementation and the existing insert function, then inspect how settings_list_set and settings_list_remove would fit with them. Confirm the intended return values for replacement and removal, and verify that both functions are exposed consistently with the existing settings API.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- rust
- Domain
- api
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100