microsoft / sarif-sdk

.NET code and supporting files for working with the 'Static Analysis Results Interchange Format' (SARIF, see https://github.com/oasis-tcs/sarif-spec)
Other
191 stars 88 forks source link

Lazy enumerated artifact #2754

Closed scottoneil-ms closed 1 week ago

scottoneil-ms commented 6 months ago

This change dusts off the "Lazy enumerated artifact" change and merges it with the Peekable stream code.

Caveat: While I looked closely at the conflicts (in EnumeratedArtifact), and believe I did the right things, I haven't re-approached this code from a global perspective beyond verifying the UTs pass.

rwoll commented 1 week ago

Thank you for your PR. As part of regular tidying and triage/prioritization, closing due to staleness.