From f8072f2fa265000a2a1be8f3dbe7e04d9dfb1c91 Mon Sep 17 00:00:00 2001
From: Chen Lihui <lihui.chen@sony.com>
Date: Tue, 23 May 2023 12:22:13 +0800
Subject: [PATCH] remove nolint since ament_cpplint updated for the c++17
 header (#2198)

Signed-off-by: Chen Lihui <lihui.chen@sony.com>
---
 rclcpp/include/rclcpp/any_subscription_callback.hpp | 2 +-
 rclcpp/include/rclcpp/client.hpp                    | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/rclcpp/include/rclcpp/any_subscription_callback.hpp b/rclcpp/include/rclcpp/any_subscription_callback.hpp
index 65b29d8535..5b662536ed 100644
--- a/rclcpp/include/rclcpp/any_subscription_callback.hpp
+++ b/rclcpp/include/rclcpp/any_subscription_callback.hpp
@@ -20,7 +20,7 @@
 #include <stdexcept>
 #include <type_traits>
 #include <utility>
-#include <variant>  // NOLINT[build/include_order]
+#include <variant>
 
 #include "rosidl_runtime_cpp/traits.hpp"
 #include "tracetools/tracetools.h"
diff --git a/rclcpp/include/rclcpp/client.hpp b/rclcpp/include/rclcpp/client.hpp
index 92f2f4af8d..85b0a2d5f9 100644
--- a/rclcpp/include/rclcpp/client.hpp
+++ b/rclcpp/include/rclcpp/client.hpp
@@ -20,13 +20,13 @@
 #include <future>
 #include <memory>
 #include <mutex>
-#include <optional>  // NOLINT, cpplint doesn't think this is a cpp std header
+#include <optional>
 #include <sstream>
 #include <string>
 #include <tuple>
 #include <unordered_map>
 #include <utility>
-#include <variant>  // NOLINT
+#include <variant>
 #include <vector>
 
 #include "rcl/client.h"