From ae7af4002066896cc4690fcce7ce3bc789dce6ff Mon Sep 17 00:00:00 2001 From: Renhe Li Date: Mon, 29 Jun 2026 15:45:59 +0800 Subject: [PATCH 1/3] Add emitter:go and linter labels Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --- .github/policies/issues.triage.generated.yml | 16 ++++++++++++++++ .github/policies/prs.triage.generated.yml | 12 ++++++++++++ CONTRIBUTING.md | 2 ++ eng/config/labels.ts | 10 ++++++++++ 4 files changed, 40 insertions(+) diff --git a/.github/policies/issues.triage.generated.yml b/.github/policies/issues.triage.generated.yml index 199afa56cb..368363aee2 100644 --- a/.github/policies/issues.triage.generated.yml +++ b/.github/policies/issues.triage.generated.yml @@ -31,6 +31,8 @@ configuration: label: emitter:python - hasLabel: label: emitter:typescript + - hasLabel: + label: emitter:go - hasLabel: label: emitter:client:all - hasLabel: @@ -41,6 +43,8 @@ configuration: label: cli/psh - hasLabel: label: meta:website + - hasLabel: + label: linter then: - addLabel: label: needs-area @@ -65,6 +69,8 @@ configuration: label: emitter:python - labelAdded: label: emitter:typescript + - labelAdded: + label: emitter:go - labelAdded: label: emitter:client:all - labelAdded: @@ -75,6 +81,8 @@ configuration: label: cli/psh - labelAdded: label: meta:website + - labelAdded: + label: linter then: - removeLabel: label: needs-area @@ -100,6 +108,8 @@ configuration: label: emitter:python - labelRemoved: label: emitter:typescript + - labelRemoved: + label: emitter:go - labelRemoved: label: emitter:client:all - labelRemoved: @@ -110,6 +120,8 @@ configuration: label: cli/psh - labelRemoved: label: meta:website + - labelRemoved: + label: linter - not: or: - hasLabel: @@ -126,6 +138,8 @@ configuration: label: emitter:python - hasLabel: label: emitter:typescript + - hasLabel: + label: emitter:go - hasLabel: label: emitter:client:all - hasLabel: @@ -136,6 +150,8 @@ configuration: label: cli/psh - hasLabel: label: meta:website + - hasLabel: + label: linter then: - addLabel: label: needs-area diff --git a/.github/policies/prs.triage.generated.yml b/.github/policies/prs.triage.generated.yml index 9ed35356f1..095f9b06ed 100644 --- a/.github/policies/prs.triage.generated.yml +++ b/.github/policies/prs.triage.generated.yml @@ -53,6 +53,12 @@ configuration: then: - addLabel: label: emitter:typescript + - if: + - includesModifiedFiles: + files: [] + then: + - addLabel: + label: emitter:go - if: - includesModifiedFiles: files: @@ -92,3 +98,9 @@ configuration: then: - addLabel: label: meta:website + - if: + - includesModifiedFiles: + files: [] + then: + - addLabel: + label: linter diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index 6411c5e658..7d4d169f72 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -439,11 +439,13 @@ Area of the codebase | `emitter:autorest` | #957300 | Issues for @azure-tools/typespec-autorest emitter | | `emitter:python` | #957300 | Issues for @azure-tools/typespec-python emitter | | `emitter:typescript` | #957300 | Issues for @azure-tools/typespec-ts emitter | +| `emitter:go` | #957300 | Issues for @azure-tools/typespec-go emitter | | `emitter:client:all` | #957300 | General client emitter issues that do not involve TCGC or typespec-azure-http-specs | | `eng` | #65bfff | | | `ide` | #846da1 | Issues for Azure specific ide features | | `cli/psh` | #9EB120 | Issues for Azure CLI/PSH features | | `meta:website` | #007dc8 | TypeSpec.io updates | +| `linter` | #65bfff | Issues related to linter rules | #### issue_kinds diff --git a/eng/config/labels.ts b/eng/config/labels.ts index f34b2afb37..884ffd2ca2 100644 --- a/eng/config/labels.ts +++ b/eng/config/labels.ts @@ -35,6 +35,10 @@ export const AreaLabels = defineLabels({ color: "957300", description: "Issues for @azure-tools/typespec-ts emitter", }, + "emitter:go": { + color: "957300", + description: "Issues for @azure-tools/typespec-go emitter", + }, "emitter:client:all": { color: "957300", description: @@ -56,6 +60,10 @@ export const AreaLabels = defineLabels({ color: "007dc8", description: "TypeSpec.io updates", }, + linter: { + color: "65bfff", + description: "Issues related to linter rules", + }, }); /** @@ -68,12 +76,14 @@ export const AreaPaths: Record = { "emitter:autorest": ["packages/typespec-autorest/"], "emitter:python": ["packages/typespec-python/"], "emitter:typescript": ["packages/typespec-ts/"], + "emitter:go": [], "lib:tcgc": ["packages/typespec-client-generator-core/"], "lib:azure-http-specs": ["packages/azure-http-specs/"], ide: [], "cli/psh": [], "emitter:client:all": [], "meta:website": ["website/"], + linter: [], }; export default defineConfig({ From 68396110b756e034bed33202db12b6b72e533907 Mon Sep 17 00:00:00 2001 From: Renhe Li Date: Wed, 1 Jul 2026 11:52:16 +0800 Subject: [PATCH 2/3] Map linter label to linter rule source paths across projects Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --- .github/policies/prs.triage.generated.yml | 8 +++++++- eng/config/labels.ts | 9 ++++++++- 2 files changed, 15 insertions(+), 2 deletions(-) diff --git a/.github/policies/prs.triage.generated.yml b/.github/policies/prs.triage.generated.yml index 095f9b06ed..9edf44295c 100644 --- a/.github/policies/prs.triage.generated.yml +++ b/.github/policies/prs.triage.generated.yml @@ -100,7 +100,13 @@ configuration: label: meta:website - if: - includesModifiedFiles: - files: [] + files: + - packages/typespec-azure-core/src/rules/**/* + - packages/typespec-azure-core/test/rules/**/* + - packages/typespec-azure-resource-manager/src/rules/**/* + - packages/typespec-azure-resource-manager/test/rules/**/* + - packages/typespec-client-generator-core/src/rules/**/* + - packages/typespec-client-generator-core/test/rules/**/* then: - addLabel: label: linter diff --git a/eng/config/labels.ts b/eng/config/labels.ts index 884ffd2ca2..8dd7fd7148 100644 --- a/eng/config/labels.ts +++ b/eng/config/labels.ts @@ -83,7 +83,14 @@ export const AreaPaths: Record = { "cli/psh": [], "emitter:client:all": [], "meta:website": ["website/"], - linter: [], + linter: [ + "packages/typespec-azure-core/src/rules/", + "packages/typespec-azure-core/test/rules/", + "packages/typespec-azure-resource-manager/src/rules/", + "packages/typespec-azure-resource-manager/test/rules/", + "packages/typespec-client-generator-core/src/rules/", + "packages/typespec-client-generator-core/test/rules/", + ], }; export default defineConfig({ From 29cc94dde142519bb2d80bde5c2645271b28913a Mon Sep 17 00:00:00 2001 From: Renhe Li Date: Wed, 1 Jul 2026 11:54:42 +0800 Subject: [PATCH 3/3] Drop test/rules from linter label paths Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --- .github/policies/prs.triage.generated.yml | 3 --- eng/config/labels.ts | 3 --- 2 files changed, 6 deletions(-) diff --git a/.github/policies/prs.triage.generated.yml b/.github/policies/prs.triage.generated.yml index 9edf44295c..d5fa56ffe4 100644 --- a/.github/policies/prs.triage.generated.yml +++ b/.github/policies/prs.triage.generated.yml @@ -102,11 +102,8 @@ configuration: - includesModifiedFiles: files: - packages/typespec-azure-core/src/rules/**/* - - packages/typespec-azure-core/test/rules/**/* - packages/typespec-azure-resource-manager/src/rules/**/* - - packages/typespec-azure-resource-manager/test/rules/**/* - packages/typespec-client-generator-core/src/rules/**/* - - packages/typespec-client-generator-core/test/rules/**/* then: - addLabel: label: linter diff --git a/eng/config/labels.ts b/eng/config/labels.ts index 8dd7fd7148..df7cfccc2c 100644 --- a/eng/config/labels.ts +++ b/eng/config/labels.ts @@ -85,11 +85,8 @@ export const AreaPaths: Record = { "meta:website": ["website/"], linter: [ "packages/typespec-azure-core/src/rules/", - "packages/typespec-azure-core/test/rules/", "packages/typespec-azure-resource-manager/src/rules/", - "packages/typespec-azure-resource-manager/test/rules/", "packages/typespec-client-generator-core/src/rules/", - "packages/typespec-client-generator-core/test/rules/", ], };