Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[@azure/monitor-query] [@azure/monitor-ingestion] Update Version Info #29943

Merged
merged 1 commit into from
Jun 7, 2024
Merged

[@azure/monitor-query] [@azure/monitor-ingestion] Update Version Info #29943

merged 1 commit into from
Jun 7, 2024

Conversation

sarangan12
Copy link
Member

Packages impacted by this PR

  1. @azure/monitor-query
  2. @azure/monitor-ingestion

Issues associated with this PR

NA

Describe the problem that is addressed by this PR

Both the above packages are scheduled for next week release. All the code changes have been merged. This is to update the version required based on the prepare release.

What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?

None

Are there test cases added in this PR? (If not, why?)

None

Provide a list of related PRs (if any)

None

Checklists

  • Added impacted package name to the issue description
  • Does this PR needs any fixes in the SDK Generator?** (If so, create an Issue in the Autorest/typescript repository and link it here)
  • Added a changelog (if necessary)

@sarangan12 sarangan12 requested review from srnagar, lmolkova and a team as code owners June 7, 2024 04:55
@github-actions github-actions bot added the Monitor Monitor, Monitor Ingestion, Monitor Query label Jun 7, 2024
@azure-sdk
Copy link
Collaborator

API change check

API changes are not detected in this pull request.

KarishmaGhiya
KarishmaGhiya approved these changes Jun 7, 2024
View reviewed changes
Copy link
Member

@KarishmaGhiya KarishmaGhiya left a comment

Choose a reason for hiding this comment

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

Please update changelog too

@sarangan12 sarangan12 merged commit 1a28734 into Azure:main Jun 7, 2024
34 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@scottaddie scottaddie scottaddie approved these changes

@KarishmaGhiya KarishmaGhiya KarishmaGhiya approved these changes

@srnagar srnagar Awaiting requested review from srnagar srnagar is a code owner

@lmolkova lmolkova Awaiting requested review from lmolkova lmolkova is a code owner

@joshfree joshfree Awaiting requested review from joshfree joshfree is a code owner automatically assigned from Azure/azure-sdk-write-monitor-data-plane

@pvaneck pvaneck Awaiting requested review from pvaneck pvaneck is a code owner automatically assigned from Azure/azure-sdk-write-monitor-data-plane

@KevinBlasko KevinBlasko Awaiting requested review from KevinBlasko KevinBlasko is a code owner automatically assigned from Azure/azure-sdk-write-monitor-data-plane

@gracewilcox gracewilcox Awaiting requested review from gracewilcox gracewilcox is a code owner automatically assigned from Azure/azure-sdk-write-monitor-data-plane

@jairmyree jairmyree Awaiting requested review from jairmyree jairmyree is a code owner automatically assigned from Azure/azure-sdk-write-monitor-data-plane

@Haiying-MSFT Haiying-MSFT Awaiting requested review from Haiying-MSFT Haiying-MSFT is a code owner automatically assigned from Azure/azure-sdk-write-monitor-data-plane

Assignees
No one assigned
Labels
Monitor Monitor, Monitor Ingestion, Monitor Query
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@sarangan12 @azure-sdk @KarishmaGhiya @scottaddie