Martin Tournoij
Martin Tournoij
Yeah, only structs are supported at the moment. It assumes the response is a JS/JSON object (`{..}`). Why? Because that's what all our APIs return :-)
Yeah, don't mind accepting patches for this. Maybe as a new `ListDirs(path string, cb func()) []Dir` function which scans a directory. You can then do `reload.Do(log.Print, cb, reload.ListDirs("./tpl", reloadTpl)...)`.
Do you get an error? Or what happens when you try to use it on Windows? I don't have a Windows machine myself, so bit hard to look at it....
No @0xhex, I don't have a Windows machine to test myself, and the OP never got back with an error. What error are you getting?
Thanks! It looks like `syscall.Exec()` isn't supported on Windows: ``` func Exec(argv0 string, argv []string, envv []string) (err error) { return EWINDOWS } ``` Where `EWINDOWS` is the `not supported...
Coolio, feel free to open a PR. Not sure why that `time.Sleep()` is in there though? 🤔
I still don't have a Windows machine @RedStalker, so still waiting for a patch. In particular, need to be careful it replaces the process correctly and that no zombie processes...
Better to flag all naked returns IMHO. That's more opinionated, but as far as I know there's fairly wide agreement that they should pretty much always be avoided. That's #170.
This has definitely taken me by surprise on a few occasions; the question is: how often is it not a problem that the defer doesn't run? I fear such a...
Actually I ran it wrong >_