Fix linter failures from includes (#110)

* Fix linter failures from includes

* reorder

* Alphabetical order was fine how it was

include/ros1_bridge/builtin_interfaces_factories.hpp

@@ -15,8 +15,6 @@
 #ifndef ROS1_BRIDGE__BUILTIN_INTERFACES_FACTORIES_HPP_
 #define ROS1_BRIDGE__BUILTIN_INTERFACES_FACTORIES_HPP_
 
-#include <ros1_bridge/factory.hpp>
-
 // include ROS 1 messages
 #include <std_msgs/Duration.h>
 #include <std_msgs/Time.h>
@@ -25,11 +23,18 @@
 #include <builtin_interfaces/msg/duration.hpp>
 #include <builtin_interfaces/msg/time.hpp>
 
+#include <memory>
+#include <string>
+
+#include "ros1_bridge/factory.hpp"
+
 namespace ros1_bridge
 {
 
 std::shared_ptr<FactoryInterface>
-get_factory_builtin_interfaces(const std::string & ros1_type_name, const std::string & ros2_type_name);
+get_factory_builtin_interfaces(
+  const std::string & ros1_type_name,
+  const std::string & ros2_type_name);
 
 // conversion functions for available interfaces
 

src/builtin_interfaces_factories.cpp

@@ -12,17 +12,22 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
+#include <memory>
+#include <string>
+
 #include "rclcpp/rclcpp.hpp"
 
 // include builtin interfaces
-#include <ros1_bridge/builtin_interfaces_factories.hpp>
-#include <ros1_bridge/convert_builtin_interfaces.hpp>
+#include "ros1_bridge/builtin_interfaces_factories.hpp"
+#include "ros1_bridge/convert_builtin_interfaces.hpp"
 
 namespace ros1_bridge
 {
 
 std::shared_ptr<FactoryInterface>
-get_factory_builtin_interfaces(const std::string & ros1_type_name, const std::string & ros2_type_name)
+get_factory_builtin_interfaces(
+  const std::string & ros1_type_name,
+  const std::string & ros2_type_name)
 {
   // mapping from string to specialized template
   if (