Skip to content

Update SECURITY.md#6867

Merged
Youssef1313 merged 2 commits into
mainfrom
dev/ygerges/security
Dec 2, 2025
Merged

Update SECURITY.md#6867
Youssef1313 merged 2 commits into
mainfrom
dev/ygerges/security

Conversation

@Youssef1313

Copy link
Copy Markdown
Member

@Youssef1313 Youssef1313 requested a review from blowdart November 3, 2025 18:00

@blowdart blowdart left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

The short link goes to another out of date security.md, so there's no point in doing this until that gets fixed.

This is also too darn short, no-one will take it seriously IMO.

Either copy the full text of the corrected security.MD the aka.ms goes to, or remove the bits that refer to email only

@Youssef1313

Copy link
Copy Markdown
Member Author

The short link goes to another out of date security.md, so there's no point in doing this until that gets fixed.

Waiting on microsoft/.github#301 then.

This is also too darn short, no-one will take it seriously IMO.

In that case, shouldn't we get https://github.com/microsoft/repo-templates/blob/main/shared/SECURITY.md updated instead to the shape we think is better?

@microsoft-github-policy-service microsoft-github-policy-service Bot added needs/attention Needs maintainer attention. and removed needs/author-feedback Waiting on the original author. labels Nov 4, 2025
@Youssef1313

Copy link
Copy Markdown
Member Author

@Youssef1313 Youssef1313 marked this pull request as ready for review December 2, 2025 17:09
@Youssef1313 Youssef1313 requested a review from blowdart December 2, 2025 17:09
@microsoft-github-policy-service microsoft-github-policy-service Bot removed the needs/attention Needs maintainer attention. label Dec 2, 2025
@Youssef1313 Youssef1313 merged commit bab3be6 into main Dec 2, 2025
4 checks passed
@Youssef1313 Youssef1313 deleted the dev/ygerges/security branch December 2, 2025 17:42
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