tailscale / tailscale/tailscale-rs
kvstore: API for remove and clone
Open
Beginner friendly
Nobody has claimed this yet.
enhancement
- Dominant language
- Rust
- Stars
- 1.2k
- Forks
- 61
- Avg merge
- 2d 13h
- Merged PRs (30d)
- 22
Description
A method which effectively just does:
let result = t.get(key).cloned();
t.remove(key);
result
where the value implements Clone, for an opt-in, ergonomic way to get the usual semantics for remove.
Contributor guide
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
Start by locating the kvstore implementation and its existing get and remove APIs. Add the opt-in operation with the stated clone-and-remove return semantics, then update the relevant kvstore tests to verify both the returned value and removal.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- rust
- Domain
- database
- Issue type
- Feature
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 68/100