Skip to content

Conversation

@cleemullins
Copy link
Contributor

@cleemullins cleemullins commented Dec 15, 2025

#minor

This pull request makes minor updates to the TranscriptUtilities test helper, primarily adding clarifying comments regarding CodeQL findings and correcting a documentation comment.

  • Documentation improvements:

    • Fixed the summary comment for the TranscriptUtilities class to properly close the XML tag and improve clarity.
  • CodeQL and code safety clarifications:

    • Added comments to suppress CodeQL [SM02729] warnings in the ExtractZipFolder method, clarifying that the code is for test purposes, runs locally, does not process user input, and the framework is planned for deprecation.

Added comments in ExtractZipFolder for CodeQL [SM02729] compliance, clarifying test-only usage and noting Entier framework archiving plans. No functional changes made.
@tracyboehrer tracyboehrer added the Automation: No parity PR does not need to be applied to other languages. label Dec 15, 2025
@cleemullins
Copy link
Contributor Author

This should be resolved by #6909. Waiting for SFI scanning to complete and confirm. Closing this PR for now.

@cleemullins cleemullins deleted the users/cleemullins/MoreSFIFixes branch December 16, 2025 18:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Automation: No parity PR does not need to be applied to other languages.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants