From 2d51e2ec427c6bc332a80321fba7c7ac0024916e Mon Sep 17 00:00:00 2001 From: Harsha Simhadri Date: Thu, 26 Mar 2026 19:43:53 -0700 Subject: [PATCH 1/2] Enhance code review instructions with additional checks Added additional code review checks regarding unit tests, dependencies, and build times. --- .github/instructions.md | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/.github/instructions.md b/.github/instructions.md index 8d7d154cc..4527ed213 100644 --- a/.github/instructions.md +++ b/.github/instructions.md @@ -1,4 +1,8 @@ -When performing a code review, please check each file has a license header +When performing a code review, check that +- No unit tests were eliminated without a strong reason +- Additional dependencies introduced have strong justification +- Changes are not likely to increase build times. +- each file has a license header ``` /* * Copyright (c) Microsoft Corporation. From a7f6a67eb0c52dc75fc27486e45bd338ee4daa2e Mon Sep 17 00:00:00 2001 From: Harsha Simhadri Date: Fri, 27 Mar 2026 08:44:42 -0700 Subject: [PATCH 2/2] Update .github/instructions.md Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> --- .github/instructions.md | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/.github/instructions.md b/.github/instructions.md index 4527ed213..784882c81 100644 --- a/.github/instructions.md +++ b/.github/instructions.md @@ -1,8 +1,8 @@ -When performing a code review, check that -- No unit tests were eliminated without a strong reason -- Additional dependencies introduced have strong justification +When performing a code review, check that: +- No unit tests were eliminated without a strong reason. +- Additional dependencies introduced have a strong justification. - Changes are not likely to increase build times. -- each file has a license header +- Each file has a license header. ``` /* * Copyright (c) Microsoft Corporation.