Skip to content

Fix trailing comments#11844

Open
satvu wants to merge 1 commit into
devfrom
satvu/fix-trailing-comment
Open

Fix trailing comments#11844
satvu wants to merge 1 commit into
devfrom
satvu/fix-trailing-comment

Conversation

@satvu

@satvu satvu commented Jun 19, 2026

Copy link
Copy Markdown
Member

Issue describing the changes in this PR

CodeQL comments need to be one line only

Pull request checklist

IMPORTANT: Currently, changes must be backported to the in-proc branch to be included in Core Tools and non-Flex deployments.

  • Backporting to the in-proc branch is not required
    • Otherwise: Link to backporting PR
  • My changes do not require documentation changes
    • Otherwise: Documentation issue linked to PR
  • My changes should not be added to the release notes for the next release
    • Otherwise: I've added my notes to release_notes.md
  • My changes do not need to be backported to a previous version
    • Otherwise: Backport tracked by issue/PR #issue_or_pr
  • My changes do not require diagnostic events changes
    • Otherwise: I have added/updated all related diagnostic events and their documentation (Documentation issue linked to PR)
  • I have added all required tests (Unit tests, E2E tests)

Copilot AI review requested due to automatic review settings June 19, 2026 19:40
@satvu satvu requested a review from a team as a code owner June 19, 2026 19:40

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates CodeQL suppression/explanation comments to be single-line, aligning with the stated requirement that CodeQL comments must be one line only.

Changes:

  • Consolidated a multi-line CodeQL SSRF justification comment into a single line in PackageDownloadHandler.cs.
  • Consolidated two multi-line CodeQL SSRF justification comments into single lines in AtlasInstanceManager.cs.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
src/WebJobs.Script.WebHost/Management/LinuxSpecialization/PackageDownloadHandler.cs Converts a multi-line CodeQL suppression rationale into a single-line comment near the HEAD request.
src/WebJobs.Script.WebHost/Management/AtlasInstanceManager.cs Converts two multi-line CodeQL suppression rationales into single-line comments near HTTP request construction.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants