xml comments

Author: SimonCropp

src/DiffEngine/DiffRunner_Inline.cs

@@ -78,6 +78,8 @@ public static async Task<InlineResult> AddInlineAsync(InlinePatch patch, Cancel
     /// conflicted entry; the other framework's differing content stays pending.
     /// </para>
     /// </summary>
+    /// <param name="sourceFile">The source file holding the call site.</param>
+    /// <param name="line">The line the snapshot was recorded at, which names the queue entry.</param>
     /// <param name="memberName">
     /// The member the call site sits in. Optional, and only used where the line no longer names
     /// the entry, which is what happens once an accept inserts a literal above it.

src/DiffEngine/Inline/InlineQueue.cs

@@ -224,6 +224,8 @@ public InlineQueue Settle(string key, string? origin) =>
     /// framework's still-failing content stays reviewable. A null origin, or an entry whose
     /// variants are all unlabeled, settles the whole entry.
     /// </summary>
+    /// <param name="key">The entry's key, naming its source file and recorded line.</param>
+    /// <param name="origin">The framework moniker of the run that started passing.</param>
     /// <param name="member">
     /// The member the settled call site sits in, used only when <paramref name="key" /> matches
     /// nothing. See <see cref="FindByMember" />.

src/DiffEngine/Protocol/ViewerMessage.cs

@@ -1,10 +1,15 @@
 namespace DiffEngine;
 
 /// <summary>
-/// A request to whoever owns the inline queue. <paramref name="Key"/> identifies a queue entry for
-/// the verbs that act on one; <paramref name="Body"/> carries an <see cref="InlinePatchFile"/>
-/// payload for <see cref="ViewerVerb.Inline"/>.
+/// A request to whoever owns the inline queue.
 /// </summary>
+/// <param name="Verb">What the owner is being asked to do.</param>
+/// <param name="Key">
+/// Identifies a queue entry, for the verbs that act on one.
+/// </param>
+/// <param name="Body">
+/// An <see cref="InlinePatchFile"/> payload, for <see cref="ViewerVerb.Inline"/>.
+/// </param>
 /// <param name="Member">
 /// The member the call site sits in, on a <see cref="ViewerVerb.Settle"/>. A fallback for the key,
 /// which names a line and so stops being true once an accept inserts a literal above it. Optional,

src/Directory.Packages.props

@@ -20,7 +20,7 @@
     <PackageVersion Include="Verify.TUnit" Version="31.28.0" />
     <PackageVersion Include="SimpleInfoName" Version="3.2.0" />
     <PackageVersion Include="Verify" Version="31.28.0" />
-    <PackageVersion Include="TUnit" Version="1.65.0" />
+    <PackageVersion Include="TUnit" Version="1.65.31" />
     <PackageVersion Include="Microsoft.Testing.Extensions.CodeCoverage" Version="18.10.0" />
   </ItemGroup>
 </Project>