Closed tnierman closed 10 months ago
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: tnierman
The full list of commands accepted by this bot can be found here.
The pull request process is described here
/retest
@tnierman: all tests passed!
Full PR test history. Your PR dashboard.
Reformats and simplifies logic around the
Tool
interface:Tool
interface frompkg/tools/base/tool.go
topkg/tool/tools.go
. Keeping the definition at the root of thetools
package, where the various implementations are defined, makes the definition easier to find and reduces the number of imports needed to utilize thetoolMap
Tool.GetName()
toTool.Name()
andTool.GetExecutableName()
toTool.ExecutableName()
to conform to the Golang standard.pkg/tools/default
Default
Tool implementation, which allows us to simplify the logic around theexecutableName
in several methods.