kragniz / kragniz/python-etcd3

Remove locks prefix or make it at least configurable

Open
#353 0 comments 4 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
450
Forks
194
PR merge metrics
No merged PRs in 30d

Description

Why do you prefix the lock keys with "/locks/"?
Why is this not mentioned in the API documentation?
Why is this not configurable?

``` python
lock_prefix = '/locks/'
[...]
self.key = lock_prefix + self.name
```

Contributor guide

Open the contributing guide

Research direction

Start at the lock implementation containing the shown lock_prefix and self.key assignments. Decide whether the lock prefix should be removed or exposed as configuration, then document the resulting behavior in the API documentation. Add or update coverage for the chosen behavior before considering the issue done.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
api, backend-api-design
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.