Github user aledsage commented on the issue:

    https://github.com/apache/brooklyn-server/pull/906
  
    LGTM - merging.
    
    For YAML examples:
    * I like to always include yaml tests if the mapping from yaml to the 
underlying Java code is non-trivial. In this case it's simple enough.
    * If the dependency order were somehow changed round, so that things could 
depend on `brooklyn-camp` and thus yaml-based tests could be written next to 
the code they relate to, then I think yaml would become much more the default 
way we write unit tests.
    * Having a yaml example in the docs would certainly be useful (I agree with 
@rdowner's point about initializers being under-the-radar. Not sure if there's 
a structure in the docs that makes it obvious where to document new ones. Also 
the `brooklyn.initializers` is not an intuitive way to configure things in 
yaml, so longer term I'd like to see that change.)


---

Reply via email to