From 45a8f340d94eb69ef7c8b8528d0ff210ad82d90a Mon Sep 17 00:00:00 2001 From: Joey Perrott Date: Tue, 5 May 2020 13:07:48 -0700 Subject: [PATCH] build: increase scope of files with enforced formatting (#36940) Historically files to be formatted were added to a listing (via matchers) to be included in formatting. Instead, this change begins efforts to instead include all files in format enforcement, relying instead on an opt out methodology. PR Close #36940 --- .ng-dev/config.ts | 28 +++++++++++----------------- 1 file changed, 11 insertions(+), 17 deletions(-) diff --git a/.ng-dev/config.ts b/.ng-dev/config.ts index eb29a1bc6f..fdb64d63ae 100644 --- a/.ng-dev/config.ts +++ b/.ng-dev/config.ts @@ -52,24 +52,18 @@ const commitMessage = { const format = { 'clang-format': { 'matchers': [ - 'dev-infra/**/*.{js,ts}', - 'packages/**/*.{js,ts}', - '!packages/zone.js', - '!packages/common/locales/**/*.{js,ts}', - '!packages/common/src/i18n/available_locales.ts', - '!packages/common/src/i18n/currencies.ts', - '!packages/common/src/i18n/locale_en.ts', - 'modules/benchmarks/**/*.{js,ts}', - 'modules/playground/**/*.{js,ts}', - 'tools/**/*.{js,ts}', - '!tools/gulp-tasks/cldr/extract.js', - '!tools/public_api_guard/**/*.d.ts', - '!tools/ts-api-guardian/test/fixtures/**', - '*.{js,ts}', - '!**/node_modules/**', - '!**/dist/**', - '!**/built/**', + '**/*.{js,ts}', + // TODO: burn down format failures and remove aio and integration exceptions. + '!aio/**', + '!integration/**', + // TODO: remove this exclusion as part of IE deprecation. '!shims_for_IE.js', + // Both third_party and .yarn are directories containing copied code which should + // not be modified. + '!third_party/**', + '!.yarn/**', + // Do not format d.ts files as they are generated + '!**/*.d.ts', ] }, 'buildifier': true