Github user tbouron commented on a diff in the pull request:

    https://github.com/apache/brooklyn-server/pull/819#discussion_r138584470
  
    --- Diff: 
core/src/main/java/org/apache/brooklyn/core/entity/Attributes.java ---
    @@ -52,11 +52,15 @@
     
         // TODO these should switch to being 
TemplatedStringAttributeSensorAndConfigKey
         BasicAttributeSensorAndConfigKey<String> DOWNLOAD_URL = new 
BasicAttributeSensorAndConfigKey<String>(
    -            String.class, "download.url", "URL pattern for downloading the 
installer (will substitute things like ${version} automatically)");
    +            String.class, 
    +            "download.url", 
    +            "URL pattern for downloading the installer (will substitute 
things like ${version} automatically)");
     
         @SuppressWarnings({ "unchecked", "rawtypes" })
         BasicAttributeSensorAndConfigKey<Map<String,String>> 
DOWNLOAD_ADDON_URLS = new BasicAttributeSensorAndConfigKey(
    -            Map.class, "download.addon.urls", "URL patterns for 
downloading named add-ons (will substitute things like ${version} 
automatically)");
    +            Map.class, 
    +            "download.addon.urls", 
    +            "URL patterns for downloading named add-ons (will substitute 
things like ${version} automatically)");
    --- End diff --
    
    Shouldn't it just say `uses FreeMarker templating format` to be consistent 
with the other description below?


---

Reply via email to