Revert breaking change in the reader for ProjectEvaluationFinished

I don't understand this, so we'll need to patch it later.

For now revert it to what it was to read existing binlogs properly.

src/StructuredLogger/BinaryLogger/BuildEventArgsReader.cs

@@ -723,12 +723,11 @@ private BuildEventArgs ReadProjectEvaluationFinishedEventArgs()
 
             if (_fileFormatVersion >= 12)
             {
-                if (_fileFormatVersion < BinaryLogger.ForwardCompatibilityMinimalVersion)
+                IEnumerable globalProperties = null;
+                if (ReadBoolean())
                 {
-                    // Throw away, but need to advance past it
-                    ReadBoolean();
+                    globalProperties = ReadStringDictionary();
                 }
-                IEnumerable? globalProperties = ReadStringDictionary();
 
                 var propertyList = ReadPropertyList();
                 var itemList = ReadProjectItems();