Skip to content

Commit dca00b7

Browse files
build(devtools): disable the "Rebase PR on target branch" circleci job so that the PR that merges the unrelated history of rangle/angular-devtools can be successfully merged into angular/angular
Usually this job would ensure that all PRs in angular/angular are running against the latest changes to angular/angular. In order to merge the unrelated history of the commits in the rangle/angular-devtools repo into angular/angular, we must temporarily disable this job.
1 parent 8c04fa7 commit dca00b7

3 files changed

Lines changed: 16 additions & 14 deletions

File tree

.circleci/config.yml

Lines changed: 15 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -237,19 +237,21 @@ jobs:
237237
- checkout
238238
- save_month_to_file
239239
- init_environment
240-
- run:
241-
name: Rebase PR on target branch
242-
# After checkout, rebase on top of target branch.
243-
command: >
244-
if [[ -n "${CIRCLE_PR_NUMBER}" ]]; then
245-
# User is required for rebase.
246-
git config user.name "angular-ci"
247-
git config user.email "angular-ci"
248-
# Rebase PR on top of target branch.
249-
node .circleci/rebase-pr.js
250-
else
251-
echo "This build is not over a PR, nothing to do."
252-
fi
240+
241+
# TODO(aleksanderbodurri): renable this as soon as angular-devtools is tranfered into angular/angular
242+
# - run:
243+
# name: Rebase PR on target branch
244+
# # After checkout, rebase on top of target branch.
245+
# command: >
246+
# if [[ -n "${CIRCLE_PR_NUMBER}" ]]; then
247+
# # User is required for rebase.
248+
# git config user.name "angular-ci"
249+
# git config user.email "angular-ci"
250+
# # Rebase PR on top of target branch.
251+
# node .circleci/rebase-pr.js
252+
# else
253+
# echo "This build is not over a PR, nothing to do."
254+
# fi
253255
# This cache is saved in the build-npm-packages so that Bazel cache is also included.
254256
- restore_cache:
255257
keys:

.ng-dev/format.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@ export const format: FormatConfig = {
1212
'clang-format': {
1313
'matchers': [
1414
'**/*.{js,ts}',
15+
'!devtools/**',
1516
// TODO: burn down format failures and remove aio and integration exceptions.
1617
'!aio/**',
1718
'!integration/**',

devtools/tslint.json

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,6 @@
1414
"directive-class-suffix": true,
1515
"directive-selector": [true, "attribute", ["ng", "app"], "camelCase"],
1616
"component-selector": [true, "element", ["ng", "app"], "kebab-case"],
17-
"import-blacklist": [true, "rxjs/Rx"],
1817
"interface-name": false,
1918
"max-classes-per-file": false,
2019
"max-line-length": [true, 140],

0 commit comments

Comments
 (0)