kataras / kataras/methodoverride
A missing inject-able defaults opts for new constructor (builder)
Open
question
- Dominant language
- Go
- Stars
- 7
- Forks
- 0
- PR merge metrics
- No merged PRs in 30d
Description
Hi,
Great package!
I think, it is better to inject these defaults opts by passing it to the constructor instead of having a predefined one in the function body. In my point of view, this new function is behaving like a builder. Maybe you could provide a builder and constructor for this package.
Best regards,
Donutloop
Contributor guide
No contributing guide indexed for this repository
Assessment
This issue has not been assessed yet.