From cf7927bd7d122b187313952dd159d55241538942 Mon Sep 17 00:00:00 2001 From: Marek Gradzki Date: Tue, 21 Aug 2018 09:01:37 +0200 Subject: Mark improvements with TODO instead of FIXME Sonar reports FIXMES as issues with higher priority as TODOs. FIXMEs should only be used for known bugs, not for improvements. Also include JIRA issue ID in the TODO message. Change-Id: I47461d948421ff17532f17ecec3ddae84239511f Signed-off-by: Marek Gradzki --- .../src/main/java/io/fd/honeycomb/data/impl/Modification.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'infra/data-impl/src/main/java/io/fd/honeycomb/data/impl/Modification.java') diff --git a/infra/data-impl/src/main/java/io/fd/honeycomb/data/impl/Modification.java b/infra/data-impl/src/main/java/io/fd/honeycomb/data/impl/Modification.java index 45a207cec..b95fc981d 100644 --- a/infra/data-impl/src/main/java/io/fd/honeycomb/data/impl/Modification.java +++ b/infra/data-impl/src/main/java/io/fd/honeycomb/data/impl/Modification.java @@ -89,12 +89,12 @@ final class Modification { } com.google.common.base.Optional> getDataBefore() { - // FIXME switch to java.util.Optional when rest of ODL infra does + // TODO(HONEYCOMB-192): switch to java.util.Optional when rest of ODL infra does return com.google.common.base.Optional.fromNullable(dataCandidate.getDataBefore().orElse(null)); } com.google.common.base.Optional> getDataAfter() { - // FIXME switch to java.util.Optional when rest of ODL infra does + // TODO(HONEYCOMB-192): switch to java.util.Optional when rest of ODL infra does return com.google.common.base.Optional.fromNullable(dataCandidate.getDataAfter().orElse(null)); } -- cgit 1.2.3-korg