This I could get some feedback. The issue was when someone needs to call 
`plugin.install(...)` with arguments. I couldn't figure out a nice API design 
to let people pass arguments, so I chose this one.

for example, core chart presets could contains
```
plugins: [
  () => new BarChartPlugin().install('bar'),
  () => new PieChartPlugin().install('pie'),
]
```

[ Full content available at: 
https://github.com/apache/incubator-superset/pull/6028 ]
This message was relayed via gitbox.apache.org for devnull@infra.apache.org

Reply via email to