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?
---