You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Hi, I'd like to discuss the idea of merging build-stats and build options with additional command like string argument, containing directory we want to save events.jsonl. For example if this directory is empty we can skip logging and system will work the same way it works now, but if we set this argument we'll store events inside of it. Something like
flagSet.StringVarP(&options.DetailedStatsFolder, "detailed-stats", "ds", "", "folder to store detailed timing and performance tracking per template"),
and in code
if r.options.DetailedStatsFolder != "" {
events.InitWithConfig(&events.ScanConfig{
couple of additional minor changes are required, but I've already using such modified build and can provide a full patch if the idea will be accepted.
reacted with thumbs up emoji reacted with thumbs down emoji reacted with laugh emoji reacted with hooray emoji reacted with confused emoji reacted with heart emoji reacted with rocket emoji reacted with eyes emoji
Uh oh!
There was an error while loading. Please reload this page.
-
Hi, I'd like to discuss the idea of merging
build-statsandbuildoptions with additional command like string argument, containing directory we want to saveevents.jsonl. For example if this directory is empty we can skip logging and system will work the same way it works now, but if we set this argument we'll store events inside of it. Something likeflagSet.StringVarP(&options.DetailedStatsFolder, "detailed-stats", "ds", "", "folder to store detailed timing and performance tracking per template"),and in code
couple of additional minor changes are required, but I've already using such modified build and can provide a full patch if the idea will be accepted.
Beta Was this translation helpful? Give feedback.
All reactions