New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Use a delegate config struct for Option #7

Open

db47h opened this issue Jun 3, 2018 · 0 comments

Owner

db47h commented Jun 3, 2018 •

edited

Loading

vm.Option directly set Intsance fields, which is bad, it should use a delegate config structure.

With the recent addition of Ticker where using Option would be safe, we need cater for most use cases
before switching to a cfg struct.

For example, a ticker function might need to remove itself, or change the tick interval.

Also need to provide more control over the instance to Go code running it.

db47h added a commit that referenced this issue


          Minor fix in ticker + update Option docs

e81c572

See #7.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment