Recently linting was enabled for `monitor-opentelemetry-exporter`. Its
eslint
config enables typed linting which requires its dependencies to be build
in
order to have proper types.

This PR passes `-t @azure/monitor-opentelemetry-exporter` to `rush
build`
before linting so that its dependencies are built.

While at it, also change the `run-for-all-packages` pipeline to use
`install-run-rush.js` for consistency.
This commit is contained in:
Jeremy Meng 2024-08-15 09:58:08 -07:00 коммит произвёл GitHub
Родитель d7d5501010
Коммит a045cb9883
Не найден ключ, соответствующий данной подписи
Идентификатор ключа GPG: B5690EEEBB952194
3 изменённых файлов: 8 добавлений и 8 удалений

Просмотреть файл

@ -25,22 +25,22 @@ stages:
displayName: "Install library dependencies"
- script: |
node common/scripts/install-run-rush.js build -t @azure/eslint-plugin-azure-sdk
node eng/tools/rush-runner.js lint -p max
node common/scripts/install-run-rush.js build -t @azure/eslint-plugin-azure-sdk -t @azure/monitor-opentelemetry-exporter
node common/scripts/install-run-rush.js lint -p max
displayName: "Build ESLint Plugin and Lint All Packages"
- script: |
node eng/tools/rush-runner.js build -p max
node common/scripts/install-run-rush.js build -p max
displayName: "Build All Packages"
- script: |
node eng/tools/rush-runner.js build:test -t @azure/identity -t @azure/app-configuration -t @azure/event-hubs -t @azure-rest/synapse-access-control -t @azure/storage-blob
node common/scripts/install-run-rush.js build:test -t @azure/identity -t @azure/app-configuration -t @azure/event-hubs -t @azure-rest/synapse-access-control -t @azure/storage-blob
displayName: "Build Tests for Selected Packages"
- script: |
node eng/tools/rush-runner.js unit-test -t @azure/identity -t @azure/app-configuration -t @azure/event-hubs -p 1
node common/scripts/install-run-rush.js unit-test -t @azure/identity -t @azure/app-configuration -t @azure/event-hubs -p 1
displayName: "Run Unit Tests for Selected Packages"
- script: |
node eng/tools/rush-runner.js integration-test --only @azure/app-configuration --only @azure/storage-blob --only @azure-rest/synapse-access-control -p 1
node common/scripts/install-run-rush.js integration-test --only @azure/app-configuration --only @azure/storage-blob --only @azure-rest/synapse-access-control -p 1
displayName: "Run Integration Tests for Selected Packages"

Просмотреть файл

@ -7,6 +7,6 @@ steps:
displayName: "Install library dependencies"
- pwsh: |
node common/scripts/install-run-rush.js build -t @azure/eslint-plugin-azure-sdk
node common/scripts/install-run-rush.js build -t @azure/eslint-plugin-azure-sdk -t @azure/monitor-opentelemetry-exporter
node eng/tools/rush-runner.js lint "${{parameters.ServiceDirectory}}" -p max
displayName: "Build ESLint Plugin and Lint Libraries"

Просмотреть файл

@ -7,7 +7,7 @@ steps:
displayName: "Install library dependencies"
- pwsh: |
node common/scripts/install-run-rush.js build -t @azure/eslint-plugin-azure-sdk
node common/scripts/install-run-rush.js build -t @azure/eslint-plugin-azure-sdk -t @azure/monitor-opentelemetry-exporter
node eng/tools/rush-runner.js lint "${{parameters.ServiceDirectory}}" -p max
displayName: "Build ESLint Plugin and Lint Libraries"