Skip to content
This repository has been archived by the owner on Jul 21, 2021. It is now read-only.

Allow for custom lock names #209

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

MrWinstead
Copy link

  • Adds NewLockConstructorOption optional Lock constructor parameters
  • Adds LockWithNameBuilder Lock constructor option
  • Adds LockNameBuilderContext to provide to lock name builders
  • Adds default lock name of "lock-" rather than hard-coded
  • Adds unit test for LockNameConstructorOption

Addresses #203

* Adds NewLockConstructorOption optional Lock constructor parameters
* Adds LockWithNameBuilder Lock constructor option
* Adds LockNameBuilderContext to provide to lock name builders
* Adds default lock name of "lock-" rather than hard-coded
* Adds unit test for LockNameConstructorOption
@coveralls
Copy link

Coverage Status

Coverage increased (+0.2%) to 80.243% when pulling 12c3859 on MrWinstead:enhancement/203 into c4fab1a on samuel:master.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants