chaoskube icon indicating copy to clipboard operation
chaoskube copied to clipboard

Consider changing contructor parameters to a struct

Open linki opened this issue 6 years ago • 2 comments

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.

linki avatar Nov 07 '19 16:11 linki

What are your thoughts on functional options?

pims avatar Nov 07 '19 22:11 pims

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.

linki avatar Nov 08 '19 10:11 linki