diff --git a/src/main/java/com/google/devtools/build/skyframe/ValueWithMetadata.java b/src/main/java/com/google/devtools/build/skyframe/ValueWithMetadata.java
index 3795832..2c2110e 100644
--- a/src/main/java/com/google/devtools/build/skyframe/ValueWithMetadata.java
+++ b/src/main/java/com/google/devtools/build/skyframe/ValueWithMetadata.java
@@ -49,9 +49,11 @@
    * Builds a value entry value that has a value value, and possibly an error (constructed from its
    * children's errors).
    *
-   * <p>This is intended only for use in alternative {@code MemoizingEvaluator} implementations.
+   * <p>This is public only for use in alternative {@code MemoizingEvaluator} implementations.
    */
-  static SkyValue normal(@Nullable SkyValue value, @Nullable ErrorInfo errorInfo,
+  public static SkyValue normal(
+      @Nullable SkyValue value,
+      @Nullable ErrorInfo errorInfo,
       NestedSet<TaggedEvents> transitiveEvents) {
     Preconditions.checkState(value != null || errorInfo != null,
         "Value and error cannot both be null");
@@ -63,7 +65,6 @@
     return new ErrorInfoValue(errorInfo, value, transitiveEvents);
   }
 
-
   @Nullable SkyValue getValue() {
     return value;
   }
@@ -124,7 +125,7 @@
   }
 
   /** Implementation of {@link ValueWithMetadata} for the error case. */
-  public static final class ErrorInfoValue extends ValueWithMetadata {
+  private static final class ErrorInfoValue extends ValueWithMetadata {
 
     private final ErrorInfo errorInfo;
     private final NestedSet<TaggedEvents> transitiveEvents;