src/Sarif/Visitors/SarifCurrentToVersionOneVisitor.cs (4 lines):
	- line 51: //  TODO: We always return null here.  Is there a pattern that allows us to change the method signature to "void" ?
	- line 495: // TODO: Issue #932
	- line 702: // TODO: add a warning to the list
	- line 1159: // TODO: Ensure stackFrame.address and offset properties are being transformed correctly during v1 -> v2 conversion and vice-versa


src/Sarif/Writers/ConsoleLogger.cs (4 lines):
	- line 14: //  TODO:  We directly instantiate this logger in two classes, creating
	- line 110: // TODO we need better retrieval for locations than these defaults.
	- line 160: // TODO: FormatForVisualStudio doesn't handle
	- line 250: // TODO we need better retrieval for locations than these defaults.


src/Sarif.Driver/Sdk/CommandBase.cs (2 lines):
	- line 21: // TODO:  Removing this constructor broke the one of AbsoluteUriCommand entirely but all unit tests were passing
	- line 29: //  TODO:  What's the point of having a bunch of static methods in an abstract class?


src/WorkItems/GitHubFilingClient.cs (2 lines):
	- line 52: // TODO: Extend GitHub issue filer to add file attachments
	- line 55: // TODO: Provide helper that generates useful attachment name from filed bug.


src/Test.FunctionalTests.Sarif/SarifValidatorTests.cs (2 lines):
	- line 93: // TODO: https://github.com/Microsoft/sarif-sdk/issues/1425
	- line 167: // TODO: why does this line of code provoke a stack overflow exception?


src/WorkItems/AzureDevOpsFilingClient.cs (2 lines):
	- line 66: //TODO: Provide helper that generates useful attachment name from filed bug.
	- line 172: // TODO: Make work item kind configurable for Azure DevOps filer


src/Sarif.Multitool.Library/RewriteCommand.cs (2 lines):
	- line 94: //  TODO: validate whether "actualOutputPath" can be created.
	- line 104: //  TODO Move this into a separate class for better unit testing


src/Test.FunctionalTests.Sarif/DefaultObjectPopulatingVisitor.cs (2 lines):
	- line 248: // TODO: Have randomization logic to ensure a valid random index is generated during population.
	- line 378: // TODO: Add additional logic to randomly select one or more required subsets and populate only those.


src/Sarif.Converters/AndroidStudioConverter.cs (2 lines):
	- line 147: // TODO enable this change after current work
	- line 194: // TODO: 'file' is another entry point type. should we be doing something here?


src/Sarif.WorkItems/SarifWorkItemContext.cs (1 line):
	- line 189: // TODO: Thoughtfully instrument this location and others


src/WorkItems/WorkItemModel.cs (1 line):
	- line 11: // TODO: Provide a meaningful representation of work item state in the model.


src/TypeScript/BuildTypeScript.ps1 (1 line):
	- line 16: # TODO:  Expand on this.


src/Sarif.WorkItems/SarifWorkItemModel.cs (1 line):
	- line 73: // TODO: Provide a useful SARIF-derived discussion entry


src/Sarif.Converters/SparseReader.cs (1 line):
	- line 338: throw new XmlException("XML Reader is in invalid state"); // TODO Code Analysis Exception


src/Test.UnitTests.Sarif/Baseline2/MatchedResultsTests.cs (1 line):
	- line 141: new FirstDetectionTimeTestCase // TODO TEST CASE SHOWING CURRENT RUN WINS FOR NEW AND PREV RUN WINS FOR EXISTING


src/Sarif/RuleUtilities.cs (1 line):
	- line 149: // TODO: Once we have migrated all the our messages to follow the convention,


src/Test.EndToEnd.Baselining/BaseliningTester.cs (1 line):
	- line 280: // TODO: Make the core baselining algorithm support BaselineFilteringMode.ToIncludedArtifacts


src/Sarif.Converters/PREFastConverter.cs (1 line):
	- line 295: // TODO: need to accept this change soon


src/Sarif.Converters/SemmleQlConverter.cs (1 line):
	- line 219: // TODO: Region.ByteOffset is not being handled correctly in SemmleQlConverter. Fix this!


src/Sarif.Driver/Sdk/MultithreadedAnalyzeCommandBase.cs (1 line):
	- line 987: // TODO single-threaded write?


src/Test.UnitTests.Sarif.Driver/SarifHelpers.cs (1 line):
	- line 45: // TODO version, etc


src/Sarif.Multitool.Library/Rules/GH1004.ReviewArraysThatExceedConfigurableDefaults.cs (1 line):
	- line 12: // TODO: root location pointer is "" in the log file. Didn't I already change that to "root" (=> "<root>"?)?


src/Sarif.Driver/DriverExtensionMethods.cs (1 line):
	- line 153: //  TODO:  Is it correct to modify options in a "validate" method?


src/Test.UnitTests.Sarif.Multitool.Library/RewriteCommandTests.cs (1 line):
	- line 205: //  TODO: Verify a separate OutputFilePath works as expected


src/Test.UnitTests.Sarif/Writers/ConsoleLoggerTests.cs (1 line):
	- line 58: // TODO: we need to get rid of the (0) literal that indicates we


src/Sarif/Baseline/ResultMatching/HeuristicMatchers/PartialFingerprintResultMatcher.cs (1 line):
	- line 12: /// TODO:  Handle versioning of partial fingerprints.


src/Test.UnitTests.Sarif/Core/ReportingDescriptorTests.cs (1 line):
	- line 94: //  TODO: Add unit tests for remaining ShouldSerialize methods


src/Sarif.Converters/ContrastSecurityConverter.cs (1 line):
	- line 247: Message = new Message { Text = $"TODO: missing message construction for rule '{ruleId}'." }


src/Sarif/Visitors/SarifVersionOneToCurrentVisitor.cs (1 line):
	- line 485: // TODO: Ensure stackFrame.address and offset properties are being transformed correctly during v1 -> v2 conversion and vice-versa


src/Sarif.WorkItems/SarifWorkItemModelTransformer.cs (1 line):
	- line 19: // TODO: make this async in the next iteration of the pipeline design


src/Sarif/Baseline/ResultMatching/SarifLogResultMatcher.cs (1 line):
	- line 257: // TODO tool message strings are not currently handled


src/Sarif/Writers/SarifLogger.cs (1 line):
	- line 211: // TODO we should actually redact across the complete log file context


src/Sarif.Driver/Sdk/AnalyzeCommandBase.cs (1 line):
	- line 211: //  TODO: This seems like uninformative error output.  All these errors get squished into one generic message