hamilton icon indicating copy to clipboard operation
hamilton copied to clipboard

@config.custom for clearer/more powerful config parsing

Open elijahbenizzy opened this issue 2 years ago • 2 comments

Is your feature request related to a problem? Please describe. @config gives no insight/structure into what config items are required, as it takes in the whole thing. Its meant to be internal but exposed externally.

Describe the solution you'd like @config.custom would take in the same argument as resolver in @resolve

Describe alternatives you've considered Keeping the current one, not supporting custom functions, keeping the current one as a fallback.

Additional context See Peter Robinson's comments here: https://hamilton-opensource.slack.com/archives/C03M33QB4M8/p1682076384844779

elijahbenizzy avatar Apr 21 '23 17:04 elijahbenizzy

Timeboxed this but there's some weirdness about how we enforce the config that made me not want to deal with this yet...

elijahbenizzy avatar Apr 21 '23 17:04 elijahbenizzy

(intended) fix for the immediate issue here, this is a follow-up.

elijahbenizzy avatar Apr 21 '23 17:04 elijahbenizzy