fix: Minor bug/typo in parseWinCertStoreType that ignored the passed in parameter

[johnweldon]

converted the function to non-member, to reduce unnecessary dependencies

[ripienaar]

which dependencies do you mean here when you say reduce dependencies? With functions like certStoreNatsOptions() this feels correctly placed - the purpose of it (see below) is to act on the *Context configuration.

Not clear to me how this was actually broken - sure it ignores t string argument but it always acts on c.config.WinCertStoreType which is the only thing it's ever being called with. Was it broken?

I think the correct thing is to keep it attached to *Context and just remove the argument t string, since its only ever called with the value it was acting on anyway

So it's basically just this:

func (c *Context) parseWinCertStoreType() (certstore.StoreType, error) {
	switch c.config.WinCertStoreType {
	case "machine":
		return certstore.ParseCertStore("windowslocalmachine")
	case "user":
		return certstore.ParseCertStore("windowscurrentuser")
	default:
		return certstore.ParseCertStore(c.config.WinCertStoreType)
	}
}

[johnweldon]

I'll make that change. I could go either way, and just picked the more functional, less object oriented, approach; but the oo approach is perfectly reasonable too