Closed kolyshkin closed 4 weeks ago
@kolyshkin: all tests passed!
Full PR test history. Your PR dashboard.
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: kolyshkin, rhmdnd
The full list of commands accepted by this bot can be found here.
The pull request process is described here
The
NewDefaultConfig
method has a few issues:it has quite a few inits that are redundant.
Remove it, and add a simple init right to the test case. We can move it out to a separate function in test later if need arises.