jasonlvhit / gocron

A Golang Job Scheduling Package.
BSD 2-Clause "Simplified" License
3.46k stars 345 forks source link

Fix DoSafely #136

Closed tiger5226 closed 4 years ago

tiger5226 commented 4 years ago

Fix - DoSafely wrapped function should not take parameters, take them from the enclosing function instead.

Fix - Do not pass jobFunc params in for wrapped function with static param set of 0.

Streppel commented 4 years ago

@JohnRoesler Hum, weird. I thought it was well configured before. I just enabled GitHub actions now, so I think subsequent PRs will trigger them!