Mark TODOs for discussion with TODO(vbreuss)

Author: vbreuss

src/System.IO.Abstractions/DirectoryInfoWrapper.cs

@@ -42,7 +42,7 @@ public override void Refresh()
         /// <inheritdoc />
         public override IFileSystemInfo ResolveLinkTarget(bool returnFinalTarget)
         {
-            //TODO: Unclear how to handle this case, as no explicit `FileSystemInfoWrapper` exists
+            //TODO(vbreuss): Unclear how to handle this case, as no explicit `FileSystemInfoWrapper` exists
             //return instance.ResolveLinkTarget(returnFinalTarget);
             return null;
         }

src/System.IO.Abstractions/FileBase.Async.cs

@@ -44,12 +44,12 @@ partial class FileBase
         public abstract Task WriteAllLinesAsync(string path, IEnumerable<string> contents, Encoding encoding, CancellationToken cancellationToken);
 
         /// <inheritdoc />
-        // TODO: Check where this overload comes from!
+        // TODO(vbreuss): Check where this overload comes from!
         [Obsolete("This overload is no longer supported in the interface")]
         public abstract Task WriteAllLinesAsync(string path, string[] contents, CancellationToken cancellationToken);
 
         /// <inheritdoc />
-        // TODO: Check where this overload comes from!
+        // TODO(vbreuss): Check where this overload comes from!
         [Obsolete("This overload is no longer supported in the interface")]
         public abstract Task WriteAllLinesAsync(string path, string[] contents, Encoding encoding, CancellationToken cancellationToken);
 

src/System.IO.Abstractions/FileBase.cs

@@ -281,7 +281,7 @@ internal FileBase() { }
 #else
         /// <inheritdoc cref="IFile.Move(string,string,bool)"/>
 #endif
-        // TODO: Check how to proceed with this method, as `Move` was also implemented on .NET5
+        // TODO(vbreuss): Check how to proceed with this method, as `Move` was also implemented on .NET5
         public abstract void Move(string sourceFileName, string destFileName, bool overwrite);
 #endif
 

src/System.IO.Abstractions/FileInfoBase.cs

@@ -58,7 +58,7 @@ internal FileInfoBase() { }
 #else
         /// <inheritdoc cref="IFileInfo.MoveTo(string,bool)"/>
 #endif
-        // TODO: Check how to proceed with this method, as `Move` was also implemented on .NET5
+        // TODO(vbreuss): Check how to proceed with this method, as `Move` was also implemented on .NET5
         public abstract void MoveTo(string destFileName, bool overwrite);
 #endif
 

src/System.IO.Abstractions/FileInfoWrapper.cs

@@ -40,7 +40,7 @@ public override void Refresh()
         /// <inheritdoc />
         public override IFileSystemInfo ResolveLinkTarget(bool returnFinalTarget)
         {
-            //TODO: Unclear how to handle this case, as no explicit `FileSystemInfoWrapper` exists
+            //TODO(vbreuss): Unclear how to handle this case, as no explicit `FileSystemInfoWrapper` exists
             //return new FileSystemInfoWrapper(FileSystem, instance.ResolveLinkTarget(returnFinalTarget));
             return null;
         }

src/System.IO.Abstractions/FileSystemWatcherBase.cs

@@ -89,7 +89,7 @@ public static implicit operator FileSystemWatcherBase(FileSystemWatcher watcher)
                 return null;
             }
 
-            //TODO: Unclear how we can implement this?!?
+            //TODO(vbreuss): Unclear how we can implement this without a IFileSystem instance?!?
             return new FileSystemWatcherWrapper(null, watcher);
         }
 

src/System.IO.Abstractions/FileWrapper.Async.cs

@@ -82,15 +82,15 @@ public override Task WriteAllLinesAsync(string path, IEnumerable<string> content
         }
 
         /// <inheritdoc />
-        // TODO: Check where this overload comes from!
+        // TODO(vbreuss): Check where this overload comes from!
         [Obsolete("This overload is no longer supported in the interface")]
         public override Task WriteAllLinesAsync(string path, string[] contents, CancellationToken cancellationToken)
         {
             return File.WriteAllLinesAsync(path, contents, cancellationToken);
         }
 
         /// <inheritdoc />
-        // TODO: Check where this overload comes from!
+        // TODO(vbreuss): Check where this overload comes from!
         [Obsolete("This overload is no longer supported in the interface")]
         public override Task WriteAllLinesAsync(string path, string[] contents, Encoding encoding, CancellationToken cancellationToken)
         {