Skip to content

Commit 3739d8f

Browse files
mkArtakMSFTCopilothalter73
authored andcommitted
Added APIReviewPrinciples document (#59880)
* Added APIReviewPrinciples document * Update docs/APIReviewPrinciples.md Co-authored-by: Copilot <[email protected]> * Update docs/APIReviewProcess.md Co-authored-by: Copilot <[email protected]> * Update docs/APIReviewProcess.md Co-authored-by: Copilot <[email protected]> * Update docs/APIReviewProcess.md * Update docs/APIReviewPrinciples.md Co-authored-by: Stephen Halter <[email protected]> * Update docs/APIReviewPrinciples.md Co-authored-by: Stephen Halter <[email protected]> --------- Co-authored-by: Copilot <[email protected]> Co-authored-by: Stephen Halter <[email protected]>
1 parent 501f6e8 commit 3739d8f

File tree

2 files changed

+18
-0
lines changed

2 files changed

+18
-0
lines changed

docs/APIReviewPrinciples.md

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,14 @@
1+
# API Review Conventions and Principles
2+
3+
Welcome to the API Review Conventions and Principles document for the dotnet/aspnetcore repository. This document serves as a comprehensive guide to the conventions and principles that our team has established over time. It is designed to encapsulate our collective knowledge and best practices, ensuring consistency and quality in our API design.
4+
5+
The primary goal of this document is to provide a reference for our team and contributors, helping them to understand and apply these principles when proposing new APIs. By adhering to these guidelines, we aim to foster a collaborative and informed API review process, ultimately enhancing the robustness and usability of our APIs.
6+
7+
We encourage you to refer to this document regularly and contribute to its evolution as we continue to refine our API review process. We also encourage you to follow the [.NET framework design guidelines](https://learn.microsoft.com/dotnet/standard/design-guidelines/) for more general guidance not specific to the dotnet/aspnetcore repository.
8+
9+
10+
## Principles
11+
- Principle 1
12+
13+
## Conventions
14+
- Convention 1

docs/APIReviewProcess.md

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,10 @@ The process is visualized in the below diagram:
1919
1. The owner of the issue is now free to work on the implementation of the proposed API.
2020
1. In case during implementation changes to the original proposal are required, the review should become obsolete and the process should start from the beginning.
2121

22+
## Learnings and growth
23+
24+
Over time the team will build up certain conventions and principles, that will be useful to establish for future API reviews too. To help with this, the [API Review Principles document](/docs/APIReviewPrinciples.md) document will be used to store such principles and conventions, making them persistent over time. Eventually, it will grow into a good knowledge base that will also help newcomers to learn and be better prepared with designing their APIs.
25+
2226
## What Makes an issue/PR "ready-for-review"?
2327

2428
Before marking an issue as `api-ready-for-review`, make sure that the issue has the following:

0 commit comments

Comments
 (0)