Open lukasjarosch opened 1 year ago
My bad, on the second try it somehow worked:
jandary@mbp doratheexplorer % go run main.go
recursive variable usage found at path 'features': ${features:sandbox}
Demo setup of what I'm trying to achieve here: https://github.com/andaryjo/doratheexplorer/tree/0805d9f440cbbd06a0c20158cad2a2c13449cd5e
I'd love to see the demo setup, but the Repo seems to be private 😄
I have reviewed your demo and it did indeed not behave as expected. The recursion detection was not working properly. I've now improved the error messages and fixed up the recursion detection.
With your example you should now receive:
cannot load target data for variable ${features:sandbox}: GetPath: undefined path: [features sandbox]
This is now indeed what a user would expect at this point, don't you think?
@lukasjarosch
These changes do not seem to resolve the problem we discussed. Trying to reference a class that goes by the same name as the yaml key still throws the same error, just less helpful this time 😄Previous:
With skipper on
feat/improve-template-undefined-value-handling
: