#449 add explicit to node constructor (#484)

* Changes defines in include guards to avoid undefined behaviour

* Adds Explicit constructor to Node class (#449)

* Update cmake.yml to run tests

---------

Co-authored-by: Olek Raymond <olek.raymond@riskaware.co.uk>

Author: JustCallMeRay

.github/workflows/cmake.yml

@@ -2,9 +2,7 @@ name: CMake
 
 on:
   push:
-    branches: [ master ]
   pull_request:
-    branches: [ master ]
 
 env:
   # Customize the CMake build type here (Release, Debug, RelWithDebInfo, etc.)

include/CXXGraph/Node/Node_decl.h

@@ -43,7 +43,7 @@ class Node {
 
   Node(const std::string &, const T &data);
   // Move constructor
-  Node(const std::string &, T &&data) noexcept;
+  explicit Node(const std::string &, T &&data) noexcept;
   ~Node() = default;
   const CXXGraph::id_t &getId() const;
   const std::string &getUserId() const;