Skip to content

Add SECURITY.md and CODE_OF_CONDUCT.md#320

Open
rgregg-msft wants to merge 2 commits intomasterfrom
add-community-health-files
Open

Add SECURITY.md and CODE_OF_CONDUCT.md#320
rgregg-msft wants to merge 2 commits intomasterfrom
add-community-health-files

Conversation

@rgregg-msft
Copy link
Contributor

@rgregg-msft rgregg-msft commented Mar 17, 2026

Summary

  • Adds SECURITY.md using the standard Microsoft template, directing reporters to https://aka.ms/SECURITY.md
  • Adds CODE_OF_CONDUCT.md adopting the Microsoft Open Source Code of Conduct

These files are required by the Microsoft open source release checklist and complete the community health files needed before publishing to NuGet (tracked in #108).

Test plan

  • Verify GitHub surfaces both files in the repo's Community Standards health check

🤖 Generated with Claude Code

Copilot AI review requested due to automatic review settings March 17, 2026 19:22
Add standard Microsoft community health files required by the open
source release checklist.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@rgregg-msft rgregg-msft force-pushed the add-community-health-files branch from d6348b2 to dc7672c Compare March 17, 2026 19:25
Copy link
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 adds required community health files for an open-source release, and also modernizes the repo’s packaging/CI setup to build, test (with coverage), and produce a .NET tool NuGet package.

Changes:

  • Add SECURITY.md and CODE_OF_CONDUCT.md using Microsoft standard templates.
  • Update CI to run on Ubuntu + Windows, collect/upload code coverage on Ubuntu, and pack/upload the tool package.
  • Migrate console packaging from a .nuspec to SDK-style dotnet pack configuration (incl. MinVer), and update various docs/tests to consistently refer to “API Doctor”.

Reviewed changes

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

Show a summary per file
File Description
docs/markdown-requirements.md Replace legacy “Markdown scanner” wording with “API Doctor”.
docs/account-configuration.md Rename headings/text to “API Doctor”.
SECURITY.md Add standard Microsoft security reporting guidance.
README.md Switch build badge from AppVeyor to GitHub Actions.
LICENSE Add MIT license file.
CONTRIBUTING.md Update contribution doc wording to “API Doctor”.
CODE_OF_CONDUCT.md Add Microsoft Open Source Code of Conduct.
ApiDoctor.Validation.UnitTests/YamlParserTests.cs Update test strings to “API Doctor”.
ApiDoctor.Validation.UnitTests/ApiDoctor.Validation.UnitTests.csproj Add coverlet.collector for coverage.
ApiDoctor.DocumentationGeneration.UnitTests/ApiDoctor.DocumentationGeneration.UnitTests.csproj Add coverlet.collector for coverage.
ApiDoctor.Console.UnitTests/ApiDoctor.Console.UnitTests.csproj Add coverlet.collector for coverage.
ApiDoctor.Console/ApiDoctor.ConsoleApp.nuspec Remove legacy .nuspec packaging.
ApiDoctor.Console/ApiDoctor.ConsoleApp.csproj Configure project for dotnet tool packing and add MinVer + package metadata.
.github/workflows/dotnet.yml Add OS matrix, NuGet caching, coverage collection/upload, and pack/upload steps.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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.

2 participants