Merge pull request #38952 from nnpcYvIVl/spelling-16

Spelling fixes

src/Features/Core/Portable/EditAndContinue/DebuggingSession.cs

@@ -75,7 +75,7 @@ internal sealed class DebuggingSession : IDisposable
 
         /// <summary>
         /// The solution captured when the debugging session entered run mode (application debugging started),
-        /// or the solution which the last changes commited to the debuggee at the end of edit session were calculated from.
+        /// or the solution which the last changes committed to the debuggee at the end of edit session were calculated from.
         /// The solution reflecting the current state of the modules loaded in the debugee.
         /// </summary>
         internal Solution LastCommittedSolution { get; private set; }

src/Test/Utilities/Portable/Assert/ConditionalFactAttribute.cs

@@ -51,8 +51,8 @@ public static class ConditionalSkipReason
     public class ConditionalFactAttribute : FactAttribute
     {
         /// <summary>
-        /// This proprety exists to prevent users of ConditionalFact from accidentally putting documentation
-        /// in the Skip proprety instead of Reason. Putting it into Skip would cause the test to be unconditionally
+        /// This property exists to prevent users of ConditionalFact from accidentally putting documentation
+        /// in the Skip property instead of Reason. Putting it into Skip would cause the test to be unconditionally
         /// skipped vs. conditionally skipped which is the entire point of this attribute.
         /// </summary>
         [Obsolete("ConditionalFact should use Reason or AlwaysSkip", error: true)]
@@ -91,8 +91,8 @@ public ConditionalFactAttribute(params Type[] skipConditions)
     public class ConditionalTheoryAttribute : TheoryAttribute
     {
         /// <summary>
-        /// This proprety exists to prevent users of ConditionalFact from accidentally putting documentation
-        /// in the Skip proprety instead of Reason. Putting it into Skip would cause the test to be unconditionally
+        /// This property exists to prevent users of ConditionalFact from accidentally putting documentation
+        /// in the Skip property instead of Reason. Putting it into Skip would cause the test to be unconditionally
         /// skipped vs. conditionally skipped which is the entire point of this attribute.
         /// </summary>
         [Obsolete("ConditionalTheory should use Reason or AlwaysSkip")]

src/Test/Utilities/Portable/Compilation/ThrowingAnalyzerAssemblyLoader.cs

@@ -8,7 +8,7 @@
 namespace Microsoft.CodeAnalysis.Test.Utilities
 {
     /// <summary>
-    /// Used when a non-runtime specific loader is needed that is never intented to be used.
+    /// Used when a non-runtime specific loader is needed that is never intended to be used.
     /// </summary>
     internal sealed class ThrowingAnalyzerAssemblyLoader : AnalyzerAssemblyLoader
     {

src/Test/Utilities/Portable/Metadata/IlasmUtilities.cs

@@ -23,7 +23,7 @@ private static string GetIlasmPath()
         {
             if (ExecutionConditionUtil.IsWindowsDesktop)
             {
-                // The destkop ilasm is still necessary because a number of our tests depend on being able to 
+                // The desktop ilasm is still necessary because a number of our tests depend on being able to 
                 // emit PDB files for net modules. That feature is not available on coreclr ilasm.
                 return Path.Combine(
                     Path.GetDirectoryName(RuntimeUtilities.GetAssemblyLocation(typeof(object))),

src/Test/Utilities/Portable/ObjectReference.cs

@@ -96,7 +96,7 @@ private void ReleaseAndGarbageCollect()
         }
 
         /// <summary>
-        /// Provides the underlying strong refernce to the given action. This method is marked not be inlined, to ensure that no temporaries are left
+        /// Provides the underlying strong reference to the given action. This method is marked not be inlined, to ensure that no temporaries are left
         /// on the stack that might still root the strong reference. The caller must not "leak" the object out of the given action for any lifetime
         /// assertions to be safe.
         /// </summary>
@@ -107,7 +107,7 @@ public void UseReference(Action<T> action)
         }
 
         /// <summary>
-        /// Provides the underlying strong refernce to the given function. This method is marked not be inlined, to ensure that no temporaries are left
+        /// Provides the underlying strong reference to the given function. This method is marked not be inlined, to ensure that no temporaries are left
         /// on the stack that might still root the strong reference. The caller must not "leak" the object out of the given action for any lifetime
         /// assertions to be safe.
         /// </summary>

src/Test/Utilities/Portable/TargetFrameworkUtil.cs

@@ -23,7 +23,7 @@ public enum TargetFramework
 
         /// <summary>
         /// Eventually this will be deleted and replaced with NetStandard20. Short term this creates the "standard"
-        /// API set across destkop and coreclr 
+        /// API set across desktop and coreclr 
         /// </summary>
         Standard,
         StandardLatest,
@@ -145,7 +145,7 @@ public static ImmutableArray<MetadataReference> GetReferences(TargetFramework tf
 
             // Check to see if there are any duplicate references. This guards against tests inadvertently passing multiple copies of 
             // say System.Core to the tests and implicitly depending on the higher one to win. The few tests which actually mean to 
-            // pass multiple verisons of a DLL should manually construct the reference list and not use this helper.
+            // pass multiple versions of a DLL should manually construct the reference list and not use this helper.
             void checkForDuplicateReferences()
             {
                 var nameSet = new HashSet<string>(getNames(references), StringComparer.OrdinalIgnoreCase);

src/Tools/Source/RunTests/ITestExecutor.cs

@@ -39,7 +39,7 @@ internal TestExecutionOptions(string xunitPath, ProcDumpInfo? procDumpInfo, stri
     /// <remarks>
     /// The difference between <see cref="TestResultInfo"/>  and <see cref="TestResult"/> is the former 
     /// is specifically for the actual test execution results while the latter can contain extra metadata
-    /// about the results.  For example whether it was cached, or had diagonstic, output, etc ...
+    /// about the results.  For example whether it was cached, or had diagnostic, output, etc ...
     /// </remarks>
     internal readonly struct TestResultInfo
     {

src/Tools/Source/RunTests/Program.cs

@@ -203,7 +203,7 @@ async Task DumpProcess(Process targetProcess, string procDumpExeFilePath, string
                     var processOutput = await processInfo.Result;
 
                     // The exit code for procdump doesn't obey standard windows rules.  It will return non-zero
-                    // for succesful cases (possibly returning the count of dumps that were written).  Best 
+                    // for successful cases (possibly returning the count of dumps that were written).  Best 
                     // backup is to test for the dump file being present.
                     if (File.Exists(dumpFilePath))
                     {