cli icon indicating copy to clipboard operation
cli copied to clipboard

Add JSON wrapping generic capability

Open meatballhat opened this issue 3 years ago • 1 comments

What type of PR is this?

  • bug
  • feature

What this PR does / why we need it:

This change adds JSON wrapping generic capability to allow for wrapping of any type with JSON marshal/unmarshal so that it may provide the Generic interface. This is also useful when reading values from altsrc providers where the type information is likely at a more primitive level than what a given Generic type may allow.

Which issue(s) this PR fixes:

Fixes #1072

Special notes for your reviewer:

I'm labeling this both kind/feature and kind/bug given that I'm addressing a bug fix (#1072) with a feature.

meatballhat avatar May 06 '22 14:05 meatballhat

@dearchap hello! Sorry, I should have said something about why I marked this as "Draft" 😅 I'm not expecting feedback yet, because I'm not convinced this is working at all like I'd want it to. If I can get it fixed up I'll bring the PR out of "Draft" and re-request your review 💖

meatballhat avatar May 07 '22 02:05 meatballhat

Nope! I'll revisit in the future if it seems like a reasonable approach.

meatballhat avatar Sep 10 '22 17:09 meatballhat