You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
We need to avoid checks for empty interfaces and map[string]interface{} and instead use reflection. This has been fixed in interpreter.go but functions.go needs the same treatment.
I'd also like to investigate adding autogen'd tests using structs from the compliance tests to verify we don't miss anything.
I'm new to this jmespath implementation, but I was hoping that it didn't use reflection given the rather steep performance penalty. Are you basically moving away from type assertions to reflection across the board or only in very narrow cases?
We need to avoid checks for empty interfaces and
map[string]interface{}
and instead use reflection. This has been fixed in interpreter.go but functions.go needs the same treatment.I'd also like to investigate adding autogen'd tests using structs from the compliance tests to verify we don't miss anything.
Related: #14
The text was updated successfully, but these errors were encountered: