Skip to content

Commit bda4ad4

Browse files
committed
ci: remove appveyor and buildkite
1 parent 9ee8075 commit bda4ad4

File tree

9 files changed

+3
-109
lines changed

9 files changed

+3
-109
lines changed

.appveyor.yml

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

.buildkite/pipeline.yml

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

.github/angular-robot.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,13 +41,13 @@ merge:
4141

4242
# list of PR statuses that need to be successful
4343
requiredStatuses:
44-
- "continuous-integration/appveyor/pr"
4544
- "ci/circleci: build"
4645
- "ci/circleci: build-bazel"
4746
- "ci/circleci: install"
4847
- "ci/circleci: lint"
4948
- "ci/circleci: validate"
5049
- "ci/circleci: test"
50+
- "ci/circleci: test-win"
5151
- "ci/circleci: test-large"
5252

5353
# the comment that will be added when the merge label is added despite failing checks, leave empty or set to false to disable

appveyor-e2e.bat

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

packages/angular_devkit/build_angular/test/dev-server/works_spec_large.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ describe('Dev Server Builder', () => {
2727
await Promise.all(runs.map(r => r.stop()));
2828
});
2929

30-
it('works', async () => {
30+
fit('works', async () => {
3131
const run = await architect.scheduleTarget(target);
3232
runs.push(run);
3333
const output = await run.result as DevServerBuilderOutput;

packages/angular_devkit/build_angular/test/karma/works_spec_large.ts

Lines changed: 1 addition & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -8,11 +8,6 @@
88
import { Architect } from '@angular-devkit/architect';
99
import { createArchitect, host, karmaTargetSpec } from '../utils';
1010

11-
// TODO: replace this with an "it()" macro that's reusable globally.
12-
let linuxOnlyIt: typeof it = it;
13-
if (process.platform.startsWith('win')) {
14-
linuxOnlyIt = xit;
15-
}
1611

1712
// tslint:disable-next-line:no-big-function
1813
describe('Karma Builder', () => {
@@ -33,9 +28,7 @@ describe('Karma Builder', () => {
3328
await run.stop();
3429
});
3530

36-
// This test seems to succeed on appveyor but not terminate Karma, leaving the port used
37-
// and killing Chrome after 60s. This causes other tests that use Chrome to fail.
38-
linuxOnlyIt('fails with broken compilation', async () => {
31+
it('fails with broken compilation', async () => {
3932
host.writeMultipleFiles({
4033
'src/app/app.component.spec.ts': '<p> definitely not typescript </p>',
4134
});

packages/angular_devkit/core/src/json/schema/registry_spec.ts

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -14,14 +14,6 @@ import { addUndefinedDefaults } from './transforms';
1414

1515
describe('CoreSchemaRegistry', () => {
1616
it('works asynchronously', done => {
17-
if (process.platform.startsWith('win')) {
18-
// This test consistently fails on Windows BuildKite, but doesn't fail on local Windows
19-
// or in Appveyor. Many tests test the async behaviour of the registry, but this is the only
20-
// one that also fetches an async ref. Perhaps that is why.
21-
done();
22-
23-
return;
24-
}
2517
const registry = new CoreSchemaRegistry();
2618
registry.addPostTransform(addUndefinedDefaults);
2719
const data: any = {}; // tslint:disable-line:no-any

scripts/test.ts

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -158,18 +158,6 @@ if (process.argv.indexOf('--spec-reporter') != -1) {
158158
// Manually set exit code (needed with custom reporters)
159159
runner.onComplete((success: boolean) => {
160160
process.exitCode = success ? 0 : 1;
161-
if (process.platform.startsWith('win')) {
162-
// TODO(filipesilva): finish figuring out why this happens.
163-
// We should not need to force exit here, but when:
164-
// - on windows
165-
// - running webpack-dev-server
166-
// - with ngtools/webpack on the compilation
167-
// Something seems to hang and the process never exists.
168-
// This does not happen on linux, nor with webpack on watch mode.
169-
// Until this is figured out, we need to exit the process manually after tests finish
170-
// otherwise appveyor will hang until it timeouts.
171-
process.exit();
172-
}
173161
});
174162

175163

tests/legacy-cli/e2e_runner.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,6 @@ Error.stackTraceLimit = Infinity;
4444
*/
4545
const argv = minimist(process.argv.slice(2), {
4646
'boolean': [
47-
'appveyor',
4847
'debug',
4948
'ng-snapshots',
5049
'noglobal',

0 commit comments

Comments
 (0)