Skip to content

Rename Agent365 contracts package - #171

Merged
Harsimar Kaur (harsimar) merged 2 commits into
mainfrom
harskaur/contracts
Oct 8, 2026
Merged

Harsimar Kaur (harsimar) merged 2 commits into
mainfrom
harskaur/contracts

Conversation

@harsimar

Copy link
Copy Markdown
Member

Publish the contracts package as Microsoft.OpenTelemetry.A365.Contracts while preserving its assembly name, namespaces, and project structure.

Co-authored-by: Copilot 223556219+Copilot@users.noreply.github.com

Publish the contracts package as Microsoft.OpenTelemetry.A365.Contracts while preserving its assembly name, namespaces, and project structure.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 91feee80-75d7-41e1-b724-3c3a5a932887

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

The rename is consistently applied while compatibility-sensitive assembly and namespace identities remain unchanged.

0 open findings

What changed in this PR

Renames the Contracts NuGet package while preserving its assembly name, namespaces, and project structure.

Changes:

  • Updates the Contracts package ID and package artifact paths.
  • Adjusts dependency and consumer restoration tests for distinct package and assembly names.
File Description
PackageDependencyTests.cs Verifies dependencies use the new package ID.
Microsoft.OpenTelemetry.Package.Tests.csproj Copies the renamed package into test feeds.
ConsumerRestoreTests.cs Separates package and assembly identity assertions.
Microsoft.Agents.A365.Observability.Contracts.csproj Sets the new NuGet package ID.

🧠 Review effort: Balanced


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@harsimar
Harsimar Kaur (harsimar) marked this pull request as draft October 8, 2026 22:04
@harsimar
Harsimar Kaur (harsimar) marked this pull request as ready for review October 8, 2026 22:13
@harsimar Harsimar Kaur (harsimar) added the skip-changelog PR does not require a CHANGELOG.md entry (CI, tests, docs, internal refactor) label Oct 8, 2026
Publish the ETW package as Microsoft.OpenTelemetry.A365.Etw while preserving its assembly name, namespaces, and project structure.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 91feee80-75d7-41e1-b724-3c3a5a932887

@alexlu4250 Jianbiao Lu (alexlu4250) left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Overall looks good.

Approving with:
Non-blocking suggestion: Renaming both NuGet package IDs breaks consumers that reference the old IDs. Make the migration explicit in release notes or package documentation, including the replacement IDs: Microsoft.OpenTelemetry.A365.Contracts and Microsoft.OpenTelemetry.A365.Etw.

@harsimar
Harsimar Kaur (harsimar) merged commit 8c71ba8 into main Oct 8, 2026
11 checks passed
@harsimar
Harsimar Kaur (harsimar) deleted the harskaur/contracts branch October 8, 2026 22:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip-changelog PR does not require a CHANGELOG.md entry (CI, tests, docs, internal refactor)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants