-
Notifications
You must be signed in to change notification settings - Fork 21.9k
Add warning for Defender AV settings replacement #127604
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
|
@rakena : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change. |
|
Learn Build status updates of commit 3e58a63: ✅ Validation status: passed
For more details, please refer to the build report. |
|
|
||
| > [!NOTE] | ||
| > Before executing this code sample, you must uncomment the variables and provide appropriate values. | ||
| > Before executing this code sample, you must uncomment the variables and provide appropriate values.<br> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| > Before executing this code sample, you must uncomment the variables and provide appropriate values.<br> | |
| > Before executing this code sample, you must uncomment the variables and provide appropriate values. | |
| > [!NOTE] | ||
| > Before executing this code sample, you must uncomment the variables and provide appropriate values. | ||
| > Before executing this code sample, you must uncomment the variables and provide appropriate values.<br> | ||
| > WARNING: Deploying or updating this extension will replace existing Defender AV settings, including exclusions. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| > WARNING: Deploying or updating this extension will replace existing Defender AV settings, including exclusions. | |
| > [!WARNING] | |
| > Deploying or updating this extension will replace existing Defender AV settings, including exclusions. |
|
@msmbaldwin - Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, adding a #label:"aq-pr-triaged" |
|
Can you review this old PR and determine whether it needs to be closed or merged? @MicrosoftDocs/public-repo-pr-review-team |
There was a problem hiding this 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 adds a warning to inform users that deploying or updating the Microsoft Antimalware extension will replace existing Defender AV settings, including exclusions. This change addresses feedback from a strategic customer escalation.
- Added a warning message about replacement of existing Defender AV settings
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
|
|
||
| > [!NOTE] | ||
| > Before executing this code sample, you must uncomment the variables and provide appropriate values. | ||
| > Before executing this code sample, you must uncomment the variables and provide appropriate values.<br> |
Copilot
AI
Oct 20, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[nitpick] The <br> HTML tag is used inconsistently with the Markdown format. Consider using a blank line with > to create a line break within the NOTE block, which is more idiomatic for Markdown blockquotes.
| > Before executing this code sample, you must uncomment the variables and provide appropriate values.<br> | |
| > Before executing this code sample, you must uncomment the variables and provide appropriate values. | |
| > |
Added a warning about replacing existing Defender AV settings when deploying or updating the extension.
This is based on customer escalation (strategic customer Mediterranean Shipping) and requested by CXP ACE team.