Skip to content

Azure Monitor Exporter: Rename#16056

Merged
rajkumar-rangaraj merged 13 commits intomasterfrom
tilee/azure_monitor_rename
Oct 23, 2020
Merged

Azure Monitor Exporter: Rename#16056
rajkumar-rangaraj merged 13 commits intomasterfrom
tilee/azure_monitor_rename

Conversation

@TimothyMothra
Copy link
Copy Markdown

@TimothyMothra TimothyMothra commented Oct 17, 2020

#15943

This PR renames the project, and in-code namespaces from "OpenTelemetry.Exporter.AzureMonitor" to "Microsoft.Azure.Monitor.OpenTelemetry.Exporter".

TODO

  • need to rename the directories, but first I wanted to make sure these changes fully compile and pass the build system.
  • need to resolve conflicts. Raj has several other PRs in progress.

@TimothyMothra
Copy link
Copy Markdown
Author

PR is currently blocked.

Generated code is not up to date. You may need to run eng\scripts\Update-Snippets.ps1 or sdk\storage\generate.ps1 or eng\scripts\Export-API.ps1

@rajkumar-rangaraj rajkumar-rangaraj merged commit 9fc81ef into master Oct 23, 2020
@rajkumar-rangaraj rajkumar-rangaraj deleted the tilee/azure_monitor_rename branch October 23, 2020 08:30
@pakrym
Copy link
Copy Markdown
Contributor

pakrym commented Oct 23, 2020

@TimothyMothra
Copy link
Copy Markdown
Author

@pakrym Yes, i'll create a follow up PR this morning.
I asked Raj to help me get this merged so we'd stop having merge conflicts in our projects.

weshaggard added a commit to Azure/azure-sdk that referenced this pull request Oct 23, 2020
weshaggard added a commit to Azure/azure-sdk that referenced this pull request Oct 23, 2020
annelo-msft pushed a commit to annelo-msft/azure-sdk-for-net that referenced this pull request Feb 17, 2021
* rename

* resolve merge conflicts

* rename, shorter project name

* merge conflicts

* Update Readme

* Rename folders

* Rename sln

* Update ci.yml

* API change

* Update API

Co-authored-by: Rajkumar Rangaraj <rajrang@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants