Merge remote-tracking branch 'origin/main' into dev/amauryleve/artifact-naming-service

Author: Evangelink

.editorconfig

@@ -301,7 +301,7 @@ dotnet_diagnostic.IDE0082.severity = warning
 dotnet_diagnostic.IDE0083.severity = warning
 
 # IDE0090: Use 'new(...)'
-dotnet_diagnostic.IDE0090.severity = warning
+dotnet_diagnostic.IDE0090.severity = none
 
 # IDE0100: Remove unnecessary equality operator
 dotnet_diagnostic.IDE0100.severity = warning
@@ -342,8 +342,11 @@ dotnet_diagnostic.IDE0270.severity = warning
 # IDE0280: Use 'nameof'
 dotnet_diagnostic.IDE0280.severity = warning
 
+# IDE0305: Simplify collection initialization
+dotnet_diagnostic.IDE0305.severity = none
+
 # IDE0370: Remove unnecessary suppression
-dotnet_diagnostic.IDE0370.severity = warning
+dotnet_diagnostic.IDE0370.severity = none
 
 # IDE1005: Use conditional delegate call
 dotnet_diagnostic.IDE1005.severity = warning

.gitattributes

@@ -66,3 +66,5 @@
 #*.PDF   diff=astextplain
 #*.rtf   diff=astextplain
 #*.RTF   diff=astextplain
+
+.github/workflows/*.lock.yml linguist-generated=true merge=ours

.github/ISSUE_TEMPLATE/bug-report.md

@@ -10,6 +10,13 @@ labels: [bug, need-triage]
 A clear and concise description of what the bug is and which version it impacts.
 -->
 
+## Version used
+
+<!--
+The MSTest or Microsoft.Testing.Platform version used that has the bug.
+Ensure you try the latest version before opening the bug.
+-->
+
 ## Steps To Reproduce
 
 <!--

.github/aw/actions-lock.json

@@ -0,0 +1,44 @@
+{
+  "entries": {
+    "actions/checkout@v6.0.2": {
+      "repo": "actions/checkout",
+      "version": "v6.0.2",
+      "sha": "de0fac2e4500dabe0009e67214ff5f5447ce83dd"
+    },
+    "actions/download-artifact@v8": {
+      "repo": "actions/download-artifact",
+      "version": "v8",
+      "sha": "3e5f45b2cfb9172054b4087a40e8e0b5a5461e7c"
+    },
+    "actions/github-script@v8": {
+      "repo": "actions/github-script",
+      "version": "v8",
+      "sha": "ed597411d8f924073f98dfc5c65a23a2325f34cd"
+    },
+    "actions/github-script@v9": {
+      "repo": "actions/github-script",
+      "version": "v9",
+      "sha": "373c709c69115d41ff229c7e5df9f8788daa9553"
+    },
+    "actions/upload-artifact@v7": {
+      "repo": "actions/upload-artifact",
+      "version": "v7",
+      "sha": "043fb46d1a93c77aae656e7c1c64a875d1fc6a0a"
+    },
+    "github/gh-aw-actions/setup-cli@v0.68.3": {
+      "repo": "github/gh-aw-actions/setup-cli",
+      "version": "v0.68.3",
+      "sha": "ba90f2186d7ad780ec640f364005fa24e797b360"
+    },
+    "github/gh-aw-actions/setup@v0.68.3": {
+      "repo": "github/gh-aw-actions/setup",
+      "version": "v0.68.3",
+      "sha": "ba90f2186d7ad780ec640f364005fa24e797b360"
+    },
+    "super-linter/super-linter@v8.5.0": {
+      "repo": "super-linter/super-linter",
+      "version": "v8.5.0",
+      "sha": "61abc07d755095a68f4987d1c2c3d1d64408f1f9"
+    }
+  }
+}

.github/copilot-instructions.md

@@ -26,17 +26,23 @@ You MUST minimize adding public API surface area but any newly added public API
 
 ## Localization Guidelines
 
-Anytime you add a new localization resource, you MUST:
+When making change to resource files, you MUST:
 
-- Add a corresponding entry in the localization resource file.
-- Add an entry in all `*.xlf` files related to the modified `.resx` file.
-- Do not modify existing entries in `*.xlf` files unless you are also modifying the corresponding `.resx` file.
+- Add a corresponding entry in the resource file (`.resx`).
+- NEVER manually modify `*.xlf` files. Instead, build the project to automatically generate the corresponding `*.xlf` files.
+
+## Public API guidelines
+
+- Public API for MSTest and Microsoft.Testing.Platform MUST NOT use `init` accessors.
+  - Exception: Existing APIs in Microsoft.Testing.Platform, because changing them right now would be a breaking change. However, we MUST NOT introduce **new** APIs using `init` accessors.
+  - IMPORTANT: Make sure to apply this rule strictly both during PR review and when working on code changes.
 
 ## Testing Guidelines
 
 - Tests for MTP and MSTest analyzers MUST use MSTest.
 - Unit tests for MSTest MUST use the internal test framework defined in [`TestFramework.ForTestingMSTest`](../test/Utilities/TestFramework.ForTestingMSTest).
 - All assertions must be written using FluentAssertions style of assertion.
+- When running acceptance tests, you must first run `./build.sh -pack`
 
 ## Pull Request guidelines
 

.github/copilot/settings.json

@@ -0,0 +1,23 @@
+{
+  "extraKnownMarketplaces": {
+    "dotnet-arcade-skills": {
+      "source": {
+        "source": "github",
+        "repo": "dotnet/arcade-skills"
+      }
+    },
+    "dotnet-skills": {
+      "source": {
+        "source": "github",
+        "repo": "dotnet/skills"
+      }
+    }
+  },
+  "enabledPlugins": {
+    "dotnet-dnceng@dotnet-arcade-skills": true,
+    "dotnet@dotnet-skills": true,
+    "dotnet-msbuild@dotnet-skills": true,
+    "dotnet-nuget@dotnet-skills": true,
+    "dotnet-test@dotnet-skills": true
+  }
+}

.github/policies/LabelManagement.IssueOpened.yml

@@ -13,13 +13,15 @@ configuration:
           - payloadType: Issues
           - isAction:
               action: Opened
-          - and:
-            - not:
-                activitySenderHasPermission:
-                  permission: Admin
-            - not:
-                activitySenderHasPermission:
-                  permission: Write
+          - not:
+              activitySenderHasPermission:
+                permission: Admin
+          - not:
+              activitySenderHasPermission:
+                permission: Write
+          - not:
+              isActivitySender:
+                user: github-actions[bot]
         then:
           - addLabel:
               label: 'Needs: Triage :mag:'

.github/policies/PullRequestIssueManagement.yml

@@ -0,0 +1,28 @@
+id: 
+name: GitOps.PullRequestIssueManagement
+description: GitOps.PullRequestIssueManagement primitive
+owner: 
+resource: repository
+disabled: false
+where: 
+configuration:
+  resourceManagementConfiguration:
+
+    eventResponderTasks:
+
+    - description: Auto-approve maestro PRs
+      triggerOnOwnActions: false
+      if:
+      - payloadType: Pull_Request
+      - isPullRequest
+      - isActivitySender:
+          user: dotnet-maestro[bot]
+          issueAuthor: False
+      - isAction:
+          action: Opened
+      then:
+      - approvePullRequest:
+          comment: Auto-approve
+
+onFailure: 
+onSuccess:

.github/policies/ScheduledSearch.MarkNoRecentActivity.yml

@@ -35,7 +35,7 @@ configuration:
               label: "State: No Recent Activity"
           - addReply:
               reply: >-
-                Hello @${issueAuthor},
+                Hello ${issueAuthor},
 
                 This pull request has been automatically marked as stale because it has been marked as requiring author feedback but has not had any activity for **7 days**. It will be closed if no further activity occurs **within 7 days of this comment**.
       - description: >-
@@ -65,8 +65,8 @@ configuration:
               label: "State: No Recent Activity"
           - addReply:
               reply: >-
-                Hello @${issueAuthor},
+                Hello ${issueAuthor},
 
                 This issue has been automatically marked as stale because it has been marked as requiring author feedback but has not had any activity for **7 days**. It will be closed if no further activity occurs **within 7 days of this comment**.
 onFailure:
-onSuccess:
+onSuccess:

.github/release.yml

@@ -0,0 +1,8 @@
+changelog:
+  exclude:
+    authors:
+      - dotnet-bot
+      - dotnet-maestro
+      - dotnet-maestro[bot]
+      - dependabot
+      - dependabot[bot]