Skip to content

chore: suppress node warnings#274905

Merged
rzhao271 merged 2 commits intomainfrom
rzhao271/ethical-penguin
Nov 4, 2025
Merged

chore: suppress node warnings#274905
rzhao271 merged 2 commits intomainfrom
rzhao271/ethical-penguin

Conversation

@rzhao271
Copy link
Copy Markdown
Collaborator

@rzhao271 rzhao271 commented Nov 3, 2025

Suppresses the warnings now that we have tried to resolve them.

Copilot AI review requested due to automatic review settings November 3, 2025 22:47
@rzhao271 rzhao271 enabled auto-merge (squash) November 3, 2025 22:47
@rzhao271 rzhao271 added this to the October 2025 milestone Nov 3, 2025
@rzhao271 rzhao271 self-assigned this Nov 3, 2025
@rzhao271 rzhao271 requested a review from deepak1556 November 3, 2025 22:47
@vs-code-engineering
Copy link
Copy Markdown
Contributor

vs-code-engineering Bot commented Nov 3, 2025

📬 CODENOTIFY

The following users are being notified based on files changed in this PR:

@lszomoru

Matched files:

  • build/azure-pipelines/win32/product-build-win32.yml

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

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 configures SDL (Security Development Lifecycle) suppression settings for the Windows build pipeline and updates the Guardian documentation reference URL.

  • Adds SDL suppression configuration to the Windows product build pipeline
  • Updates the Guardian help URI from "baselines" to "suppressions" documentation

Reviewed Changes

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

File Description
build/azure-pipelines/win32/product-build-win32.yml Adds SDL suppression configuration pointing to the Guardian suppression file
.config/guardian/.gdnsuppress Updates help URI to point to the correct "suppressions" documentation page

Copy link
Copy Markdown
Collaborator

@deepak1556 deepak1556 left a comment

Choose a reason for hiding this comment

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

Should we run a build to confirm the suppressions are picked up ?

@rzhao271 rzhao271 merged commit 18d828e into main Nov 4, 2025
28 checks passed
@rzhao271 rzhao271 deleted the rzhao271/ethical-penguin branch November 4, 2025 03:45
@rzhao271
Copy link
Copy Markdown
Collaborator Author

rzhao271 commented Nov 4, 2025

Will check on Friday or during debt week.

@vs-code-engineering vs-code-engineering Bot locked and limited conversation to collaborators Dec 19, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants