Skip to content

Commit d3fe294

Browse files
Remove core ci from ado (#10307)
Final step in migration to github action for typespec core. Removing the old ADO CI.
1 parent a385cfd commit d3fe294

6 files changed

Lines changed: 7 additions & 170 deletions

File tree

eng/common/config/area.ts

Lines changed: 0 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -48,21 +48,4 @@ export const CIRules = {
4848
CSharp: [...all, ...isolatedEmitters, ...AreaPaths["emitter:client:csharp"], ".editorconfig"],
4949
Java: [...all, ...isolatedEmitters, ...AreaPaths["emitter:client:java"], ".editorconfig"],
5050
Python: [...all, ...isolatedEmitters, ...AreaPaths["emitter:client:python"], ".editorconfig"],
51-
52-
Core: [
53-
"**/*",
54-
"!.prettierignore", // Prettier is already run as its dedicated CI(via github action)
55-
"!.prettierrc.json",
56-
"!cspell.yaml", // CSpell is already run as its dedicated CI(via github action)
57-
"!eslint.config.json", // Eslint is already run as its dedicated CI(via github action)
58-
"!.chronus/**/*", // Used across emitters
59-
...ignore(isolatedEmitters),
60-
...ignore(AreaPaths["emitter:client:csharp"]),
61-
...ignore(AreaPaths["emitter:client:java"]),
62-
...ignore(AreaPaths["emitter:client:python"]),
63-
],
6451
};
65-
66-
function ignore(paths: string[]) {
67-
return paths.map((x) => `!${x}`);
68-
}

eng/common/pipelines/ci.yml

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -59,9 +59,3 @@ extends:
5959
parameters:
6060
DependsOn: InitStage
6161
Condition: eq('true', stageDependencies.InitStage.outputs['InitJob.InitStep.RunPython'])
62-
63-
# Run core stages if RunCore == true
64-
- template: /eng/tsp-core/pipelines/stages/ci-stages.yml
65-
parameters:
66-
DependsOn: InitStage
67-
Condition: eq('true', stageDependencies.InitStage.outputs['InitJob.InitStep.RunCore'])

eng/common/scripts/utils/find-area-changed.test.ts

Lines changed: 7 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -38,17 +38,6 @@ describe("paths that should trigger python CI", () => {
3838
});
3939
});
4040

41-
describe("paths that should trigger Core CI", () => {
42-
it.each([
43-
"packages/compiler/package.json",
44-
"packages/http/package.json",
45-
"packages/openapi3/package.json",
46-
])("%s", (path) => {
47-
const areas = findAreasChanged([path]);
48-
expect(areas).toEqual(["Core"]);
49-
});
50-
});
51-
5241
describe("paths that should trigger all isolated packages", () => {
5342
it.each([
5443
"eng/emitters/pipelines/templates/jobs/detect-api-changes.yml",
@@ -59,22 +48,22 @@ describe("paths that should trigger all isolated packages", () => {
5948
});
6049
});
6150

62-
it("Should return a combination of core and isolated packages", () => {
51+
it("Should return a combination of isolated packages", () => {
6352
const areas = findAreasChanged([
6453
"packages/http-client-csharp/src/constants.ts",
6554
"packages/http-client-java/src/emitter.ts",
6655
"packages/http-client-python/src/emitter.ts",
6756
"packages/compiler/package.json",
6857
]);
69-
expect(areas).toEqual(["CSharp", "Java", "Python", "Core"]);
58+
expect(areas).toEqual(["CSharp", "Java", "Python"]);
7059
});
7160

72-
it("Should return CSharp, Core and Java if .editorconfig is changed", () => {
61+
it("Should return CSharp, Java and Python if .editorconfig is changed", () => {
7362
const areas = findAreasChanged([".editorconfig"]);
74-
expect(areas).toEqual(["CSharp", "Java", "Python", "Core"]);
63+
expect(areas).toEqual(["CSharp", "Java", "Python"]);
7564
});
7665

77-
it("Should not return Core for .prettierignore, .prettierrc.json, cspell.yaml, eslint.config.json", () => {
66+
it("Should not trigger CI for .prettierignore, .prettierrc.json, cspell.yaml, eslint.config.json alone", () => {
7867
const areas = findAreasChanged([
7968
".prettierignore",
8069
".prettierrc.json",
@@ -87,7 +76,7 @@ it("Should not return Core for .prettierignore, .prettierrc.json, cspell.yaml, e
8776
expect(areas).toEqual(["CSharp", "Java", "Python"]);
8877
});
8978

90-
it("should return Core for random files at the root", () => {
79+
it("should not trigger any CI for random files at the root", () => {
9180
const areas = findAreasChanged(["some.file", "file/in/deep/directory"]);
92-
expect(areas).toEqual(["Core"]);
81+
expect(areas).toEqual([]);
9382
});

eng/tsp-core/pipelines/jobs/build-and-test.yml

Lines changed: 0 additions & 40 deletions
This file was deleted.

eng/tsp-core/pipelines/jobs/website.yml

Lines changed: 0 additions & 11 deletions
This file was deleted.

eng/tsp-core/pipelines/stages/ci-stages.yml

Lines changed: 0 additions & 78 deletions
This file was deleted.

0 commit comments

Comments
 (0)