-
Notifications
You must be signed in to change notification settings - Fork 121
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.
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
Consider changing contructor parameters to a struct #157
Labels
Comments
What are your thoughts on functional options? |
I'm not a big fan of them but I also don't have much experience with them. From your linked article they provide exactly what I would like so I wouldn't mind giving it a try here. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The number of arguments for
Chaoskube.New()
has increased to a point where it becomes annoying to use.We could switch to a struct to have order-independent arguments by Name. It would also allow us to leave out keys when we want the default value.
The text was updated successfully, but these errors were encountered: