Skip to content
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

Docs: Update repo templates #4677

Merged
merged 5 commits into from
May 21, 2024

Conversation

StephanTLavavej
Copy link
Member

This satisfies an upcoming requirement (internal link) to update our "repo templates" from microsoft/repo-templates. I'm not exactly sure how they're going to enforce compliance, so instead of altering the files to conform to our awesome strict conventions, I'm adding binary-identical copies with exemptions from validation.

  • Delete old SECURITY.md.
  • Copy exactly the latest version of SECURITY.md.
  • Copy exactly the latest version of CODE_OF_CONDUCT.md.
  • Centralize on the newly added CODE_OF_CONDUCT.md.
  • Skip validation of CODE_OF_CONDUCT.md and SECURITY.md.
    • CODE_OF_CONDUCT.md contains LF line endings.
    • SECURITY.md contains LF line endings, 1 line with trailing whitespace, and 7 lines with more than 120 columns.

@StephanTLavavej StephanTLavavej added the documentation Related to documentation or comments label May 14, 2024
@StephanTLavavej StephanTLavavej requested a review from a team as a code owner May 14, 2024 22:18
@StephanTLavavej StephanTLavavej self-assigned this May 17, 2024
@StephanTLavavej
Copy link
Member Author

I'm mirroring this to the MSVC-internal repo - please notify me if any further changes are pushed.

@StephanTLavavej StephanTLavavej merged commit 63354c3 into microsoft:main May 21, 2024
39 checks passed
@StephanTLavavej StephanTLavavej deleted the update-repo-templates branch May 21, 2024 00:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Related to documentation or comments
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet

2 participants