src/Microsoft.TestPlatform.VsTestConsole.TranslationLayer/VsTestConsoleRequestSender.cs (4 lines):
	- line 471: // TODO (copoiena): When sharing the test host between test discovery and test
	- line 575: // TODO (copoiena): When sharing the test host between test discovery and test
	- line 918: // TODO : Payload for ProtocolError needs to finalized.
	- line 961: // TODO : Payload for ProtocolError needs to finalized.


src/Microsoft.TestPlatform.TestHostProvider/Hosting/DotnetTestHostManager.cs (3 lines):
	- line 210: // TODO: Throw an exception when we get 0 or more than 1 source, that explains what happened, instead of .Single throwing a generic exception?
	- line 376: // TODO: This version check is a hack, when the target framework is figured out it tries to unify to a single common framework
	- line 450: // TODO this may be required in ASP.net, requires validation


eng/common/pipeline-logging-functions.ps1 (3 lines):
	- line 6: $script:loggingCommandEscapeMappings = @( # TODO: WHAT ABOUT "="? WHAT ABOUT "%"?
	- line 12: # TODO: BUG: Escape % ???
	- line 13: # TODO: Add test to verify don't need to escape "=".


src/Microsoft.TestPlatform.CrossPlatEngine/Execution/BaseRunTests.cs (2 lines):
	- line 237: // TODO : this does not crash the process currently because of the job queue.
	- line 609: // TODO: enable the below once runsettings is in.


src/vstest.console/CommandLine/InferHelper.cs (2 lines):
	- line 103: // TODO Log to console and client.
	- line 136: // TODO What else to do with appx, js and other?


src/Microsoft.TestPlatform.Common/Filtering/FilterExpressionWrapper.cs (1 line):
	- line 53: // TODO: surface an error message to suer.


src/Microsoft.TestPlatform.Common/DataCollection/DataCollectionManager.cs (1 line):
	- line 121: // TODO : change IMessageSink and use IMessageLogger instead.


src/Microsoft.TestPlatform.Build/Tasks/VSTestTask.cs (1 line):
	- line 234: // TODO log arguments in task


src/Microsoft.TestPlatform.CrossPlatEngine/Execution/RunTestsWithTests.cs (1 line):
	- line 126: // TODO: Fill this in with the right extension value.


src/Microsoft.TestPlatform.Common/ExtensionFramework/TestExecutorExtensionManager.cs (1 line):
	- line 188: // TODO: This can be optimized - The base class's populate map would be called repeatedly for the same extension assembly.


src/Microsoft.TestPlatform.Extensions.BlameDataCollector/ProcessCodeMethods.cs (1 line):
	- line 37: // TODO: do not suspend on Mac and Linux, this prevents the process from being dumped when we use the net client dumper, checking if we can post a different signal


src/testhost.x86/DefaultEngineInvoker.cs (1 line):
	- line 218: //TODO: should there be a warning / error in this case, on windows and linux we are most likely not started by this PID 0, because it's Idle process on Windows, and Swapper on Linux, and similarly in docker


eng/SourceBuild.props (1 line):
	- line 18: <!-- TODO: Remove once all patches have been removed. -->


src/vstest.console/Internal/ConsoleLogger.cs (1 line):
	- line 206: // TODO Get changes from https://github.com/Microsoft/vstest/pull/1111/


src/vstest.console/TestPlatformHelpers/TestRequestManager.cs (1 line):
	- line 455: // TODO (copoiena): Collect metrics ?


src/Microsoft.TestPlatform.CommunicationUtilities/TestRequestSender.cs (1 line):
	- line 746: // TODO: Use factory to get the communication endpoint. It will abstract out the type of communication endpoint like socket, shared memory or named pipe etc.,


src/Microsoft.TestPlatform.CrossPlatEngine/TestSession/TestSessionPool.cs (1 line):
	- line 97: // TODO (copoiena): What happens if some request is running for the current session ?


src/Microsoft.TestPlatform.CommunicationUtilities/Interfaces/IDataCollectionTestCaseEventHandler.cs (1 line):
	- line 11: // TODO : Similar interfaces exist, need redesign.


src/Microsoft.TestPlatform.VsTestConsole.TranslationLayer/ConsoleParameters.cs (1 line):
	- line 44: /// TODO: Remove the #if when project is targeted to netstandard2.0


src/Microsoft.TestPlatform.Extensions.TrxLogger/Utility/Collection.cs (1 line):
	- line 94: // TODO: Consider putting check for null to derived classes.


src/Microsoft.TestPlatform.ObjectModel/TraitCollection.cs (1 line):
	- line 25: // TODO: Fix this with proper resourcing for UWP and Win 8.1 Apps


src/Microsoft.TestPlatform.Utilities/XmlUtilities.cs (1 line):
	- line 60: // TODO: There isn't an equivalent API to SecurityElement.Escape in Core yet.


src/Microsoft.TestPlatform.CommunicationUtilities/SocketCommunicationManager.cs (1 line):
	- line 167: // TODO: pass cancellation token, if user cancels the operation, so we don't wait 50 secs to connect


src/Microsoft.TestPlatform.CrossPlatEngine/TestEngine.cs (1 line):
	- line 310: // TODO (copoiena): For now we don't support data collection alongside test


src/Microsoft.TestPlatform.PlatformAbstractions/common/System/ProcessHelper.cs (1 line):
	- line 100: // TODO: Enable logging here, and consider wrapping Win32Exception into another that shows the path of the process.


src/Microsoft.TestPlatform.Client/TestPlatform.cs (1 line):
	- line 43: // TODO: This is not the right way to force initialization of default extensions.


src/Microsoft.TestPlatform.CommunicationUtilities/DataCollectionTestCaseEventSender.cs (1 line):
	- line 44: // TODO : Re-factor to pass the instance as singleton.


src/Microsoft.TestPlatform.CrossPlatEngine/Discovery/DiscovererEnumerator.cs (1 line):
	- line 337: // TODO : Enable this when RunSettings is enabled.


src/Microsoft.TestPlatform.ObjectModel/RunSettings/TestRunSettings.cs (1 line):
	- line 38: /// TODO: Communicate to Chutzpah and fix it


src/Microsoft.TestPlatform.CrossPlatEngine/Client/ProxyOperationManager.cs (1 line):
	- line 353: // TODO (copoiena): If called and testhost is already running, we should restart.


src/Microsoft.TestPlatform.Client/Execution/TestRunRequest.cs (1 line):
	- line 496: // TODO: Invoke this event in a separate thread.


src/Microsoft.TestPlatform.CrossPlatEngine/TestSession/ProxyTestSessionManager.cs (1 line):
	- line 163: // TODO (copoiena): This run settings match is rudimentary. We should refine the