Make fml/status_or.h compatible with .clang_tidy.

fml/status_or.h

@@ -31,9 +31,14 @@ namespace fml {
 template <typename T>
 class StatusOr {
  public:
+  // These constructors are intended be compatible with absl::status_or.
+  // NOLINTBEGIN(google-explicit-constructor)
+
   StatusOr(const T& value) : status_(), value_(value) {}
   StatusOr(const Status& status) : status_(status), value_() {}
 
+  // NOLINTEND(google-explicit-constructor)
+
   StatusOr(const StatusOr&) = default;
   StatusOr(StatusOr&&) = default;
 
@@ -63,13 +68,17 @@ class StatusOr {
   bool ok() const { return status_.ok(); }
 
   const T& value() const {
-    FML_CHECK(status_.ok());
-    return value_.value();
+    if (status_.ok()) {
+      return value_.value();
+    }
+    FML_LOG(FATAL) << "StatusOr::value() called on error Status";
   }
 
   T& value() {
-    FML_CHECK(status_.ok());
-    return value_.value();
+    if (status_.ok()) {
+      return value_.value();
+    }
+    FML_LOG(FATAL) << "StatusOr::value() called on error Status";
   }
 
  private: