Skip to content

[monitor] Fix pipeline issue from missing dependency#28906

Merged
xirzec merged 1 commit into
Azure:mainfrom
xirzec:fixMonitorPipelines
Mar 13, 2024
Merged

[monitor] Fix pipeline issue from missing dependency#28906
xirzec merged 1 commit into
Azure:mainfrom
xirzec:fixMonitorPipelines

Conversation

@xirzec
Copy link
Copy Markdown
Member

@xirzec xirzec commented Mar 13, 2024

Packages impacted by this PR

@azure/monitor-opentelemetry
@azure/monitor-opentelemetry-exporter

Describe the problem that is addressed by this PR

cross-env was removed from the devdeps of monitor-opentelemetry and monitor-opentelemetry-exporter which causes pipeline failures when running the test pipeline since the integration test command uses this binary.

`cross-env` was removed from the devdeps of monitor-opentelemetry and monitor-opentelemetry-exporter which causes pipeline failures when running the test pipeline since the integration test command uses this binary.
@xirzec xirzec self-assigned this Mar 13, 2024
@xirzec xirzec added Client This issue points to a problem in the data-plane of the library. Monitor Monitor, Monitor Ingestion, Monitor Query labels Mar 13, 2024
@xirzec xirzec enabled auto-merge (squash) March 13, 2024 19:04
Copy link
Copy Markdown
Member

@hectorhdzg hectorhdzg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@azure-sdk
Copy link
Copy Markdown
Collaborator

API change check

API changes are not detected in this pull request.

@xirzec xirzec merged commit 2409cde into Azure:main Mar 13, 2024
@xirzec xirzec deleted the fixMonitorPipelines branch March 13, 2024 20:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Client This issue points to a problem in the data-plane of the library. Monitor Monitor, Monitor Ingestion, Monitor Query

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants