ci: remove typescript 3.7 patches in material-unit-tests job (#34863)
Previously, we added patches to the `angular/components` repository in the `material-unit-tests` job in order to be able to run their tests with TypeScript 3.7. This is no longer needed since the components repository updated to Ts 3.7 already. Hence we can remove the patches. PR Close #34863
This commit is contained in:
parent
20419d3d49
commit
7342d4b6f0
|
@ -779,21 +779,6 @@ jobs:
|
|||
# that it's not necessary to perform a yarn install, as Bazel runs it automatically.
|
||||
name: Setting up release packages.
|
||||
command: node scripts/ci/update-deps-to-dist-packages.js ${MATERIAL_REPO_TMP_DIR}/package.json dist/packages-dist/
|
||||
- run:
|
||||
# Install TypeScript 3.7 and Tsickle 0.38.0.
|
||||
name: Setting up TypeScript 3.7 and Tsickle 0.38.0
|
||||
# TODO: remove this once the repo has been updated to use TypeScript 3.7 and
|
||||
# Tsickle 0.38.0. Note that the components repository has a Yarn resolution to ensure
|
||||
# that "dgeni-packages" uses a specific TypeScript version. This resolution causes the
|
||||
# specified TS version to be a valid candidate for the "@angular/bazel" peer dependency.
|
||||
# Ultimately, Yarn seems to use the TS version from the resolution for Angular Bazel.
|
||||
# This causes a mismatch of TypeScript versions as tsickle will use the v3.7.4 version,
|
||||
# while Angular Bazel uses any arbitrary version from the resolution (currently v3.6.4).
|
||||
# Also note that we need to explicitly update `tsickle` because `angular/components`
|
||||
# declared it as explicit dep, and their current version is not compatible with TS 3.7.
|
||||
command: |
|
||||
sed -i -E 's#(dgeni-packages/typescript": ").+"#\13.7.4"#' ${MATERIAL_REPO_TMP_DIR}/package.json
|
||||
yarn --ignore-engines --cwd ${MATERIAL_REPO_TMP_DIR} add typescript@3.7.4 tsickle@0.38.0 --dev
|
||||
- run:
|
||||
name: "Running Material unit tests"
|
||||
command: ./scripts/ci/run_angular_material_unit_tests.sh
|
||||
|
|
Loading…
Reference in New Issue