Revise SmartScreen reputation article for clarity and updates - #5928
Merged
Steven White (stevewhims) merged 3 commits intoMay 5, 2026
Conversation
Updated the SmartScreen reputation article for accuracy.
Contributor
|
Eric Lawrence (@ericlaw1979) : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change. |
Contributor
|
Learn Build status updates of commit 412f736: ✅ Validation status: passed
For more details, please refer to the build report. |
Corrected minor grammatical issues and clarified phrases regarding SmartScreen warnings and reputation.
Contributor
|
Learn Build status updates of commit fc384c1:
|
| File | Status | Preview URL | Details |
|---|---|---|---|
| hub/apps/package-and-deploy/smartscreen-reputation.md | Details |
hub/apps/package-and-deploy/smartscreen-reputation.md
- Line 68, Column 88: [Warning: hard-coded-locale - See documentation]
Link 'https://learn.microsoft.com/en-us/windows/win32/secbp/understanding-pe-signatures#security-implications' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites. - Line 69, Column 93: [Warning: hard-coded-locale - See documentation]
Link 'https://learn.microsoft.com/en-us/unified-secops/criteria' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites. - Line 68, Column 88: [Suggestion: docs-link-absolute - See documentation]
Absolute link 'https://learn.microsoft.com/en-us/windows/win32/secbp/understanding-pe-signatures#security-implications' will be broken in isolated environments. Replace with a relative link. - Line 69, Column 93: [Suggestion: docs-link-absolute - See documentation]
Absolute link 'https://learn.microsoft.com/en-us/unified-secops/criteria' will be broken in isolated environments. Replace with a relative link.
For more details, please refer to the build report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
Contributor
|
Learn Build status updates of commit 002b14d: ✅ Validation status: passed
For more details, please refer to the build report. |
Steven White (stevewhims)
approved these changes
May 5, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Updated the SmartScreen reputation article for accuracy.