-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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.
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
[AzureMonitorExporter] cleanup Changelog before release #31359
Conversation
Azure SDK standards DOES specify how PR links should be formatted.
I'll work on this change |
API change check API changes are not detected in this pull request. |
I saw a planning email from Matt earlier today. Confirmed that the target release date is Friday Oct 7th. |
Co-authored-by: Vishwesh Bankwar <vishweshbankwar@users.noreply.github.com>
Co-authored-by: Vishwesh Bankwar <vishweshbankwar@users.noreply.github.com>
Co-authored-by: Vishwesh Bankwar <vishweshbankwar@users.noreply.github.com>
Co-authored-by: Vishwesh Bankwar <vishweshbankwar@users.noreply.github.com>
Co-authored-by: Vishwesh Bankwar <vishweshbankwar@users.noreply.github.com>
Co-authored-by: Vishwesh Bankwar <vishweshbankwar@users.noreply.github.com>
Co-authored-by: Vishwesh Bankwar <vishweshbankwar@users.noreply.github.com>
Co-authored-by: Vishwesh Bankwar <vishweshbankwar@users.noreply.github.com>
Co-authored-by: Vishwesh Bankwar <vishweshbankwar@users.noreply.github.com>
Co-authored-by: Vishwesh Bankwar <vishweshbankwar@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM - it looks clean and easy to understand now.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
* [AzureMonitorExporter] cleanup Changelog before release * update links * Update sdk/monitor/Azure.Monitor.OpenTelemetry.Exporter/CHANGELOG.md Co-authored-by: Vishwesh Bankwar <vishweshbankwar@users.noreply.github.com> * Update sdk/monitor/Azure.Monitor.OpenTelemetry.Exporter/CHANGELOG.md Co-authored-by: Vishwesh Bankwar <vishweshbankwar@users.noreply.github.com> * Update sdk/monitor/Azure.Monitor.OpenTelemetry.Exporter/CHANGELOG.md Co-authored-by: Vishwesh Bankwar <vishweshbankwar@users.noreply.github.com> * Update sdk/monitor/Azure.Monitor.OpenTelemetry.Exporter/CHANGELOG.md Co-authored-by: Vishwesh Bankwar <vishweshbankwar@users.noreply.github.com> * Update sdk/monitor/Azure.Monitor.OpenTelemetry.Exporter/CHANGELOG.md Co-authored-by: Vishwesh Bankwar <vishweshbankwar@users.noreply.github.com> * Update sdk/monitor/Azure.Monitor.OpenTelemetry.Exporter/CHANGELOG.md Co-authored-by: Vishwesh Bankwar <vishweshbankwar@users.noreply.github.com> * Update sdk/monitor/Azure.Monitor.OpenTelemetry.Exporter/CHANGELOG.md Co-authored-by: Vishwesh Bankwar <vishweshbankwar@users.noreply.github.com> * Update sdk/monitor/Azure.Monitor.OpenTelemetry.Exporter/CHANGELOG.md Co-authored-by: Vishwesh Bankwar <vishweshbankwar@users.noreply.github.com> * Update sdk/monitor/Azure.Monitor.OpenTelemetry.Exporter/CHANGELOG.md Co-authored-by: Vishwesh Bankwar <vishweshbankwar@users.noreply.github.com> * Update sdk/monitor/Azure.Monitor.OpenTelemetry.Exporter/CHANGELOG.md Co-authored-by: Vishwesh Bankwar <vishweshbankwar@users.noreply.github.com> * reference latest OTel dependencies PR * Update CHANGELOG.md * Update CHANGELOG.md * Update CHANGELOG.md * Update CHANGELOG.md * added link to ApplicationInsightsSampler Co-authored-by: Vishwesh Bankwar <vishweshbankwar@users.noreply.github.com>
Reviewing the Changelog before our next release.
I'm opening this PR so our team can discuss changes.
We already verified we have the correct sections:
TODO