Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Nov 19, 2025

Addresses review feedback from PR #63 regarding non-standard copyright header in utils.py.

Changes

  • Updated copyright header in libraries/microsoft-agents-a365-observability-core/microsoft_agents_a365/observability/core/utils.py from # Copyright (c) Microsoft. All rights reserved. to the standard two-line format:
# Copyright (c) Microsoft Corporation.
# Licensed under the MIT License.

This aligns with the repository's code review guidelines requiring the standard Microsoft copyright header format for all Python files.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: nikhilNava <211831449+nikhilNava@users.noreply.github.com>
Copilot AI changed the title [WIP] Update exporter options and functionality based on feedback Fix copyright header in utils.py to use standard Microsoft format Nov 19, 2025
Copilot AI requested a review from nikhilNava November 19, 2025 20:18
@nikhilNava nikhilNava marked this pull request as ready for review November 19, 2025 20:19
@nikhilNava nikhilNava requested a review from a team as a code owner November 19, 2025 20:19
@nikhilNava nikhilNava merged commit 297bb02 into nikhilc/addExporterOptions Nov 19, 2025
1 check passed
@nikhilNava nikhilNava deleted the copilot/sub-pr-63-again branch November 19, 2025 20:19
nikhilNava added a commit that referenced this pull request Nov 20, 2025
* updates to baggage and scope

* add tests for new functionality

* fix tests

* Update libraries/microsoft-agents-a365-observability-core/microsoft_agents_a365/observability/core/invoke_agent_scope.py

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

* Update libraries/microsoft-agents-a365-observability-core/microsoft_agents_a365/observability/core/middleware/baggage_builder.py

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

* Update libraries/microsoft-agents-a365-observability-core/microsoft_agents_a365/observability/core/exporters/agent365_exporter_options.py

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

* Update libraries/microsoft-agents-a365-observability-core/microsoft_agents_a365/observability/core/exporters/agent365_exporter_options.py

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

* Update libraries/microsoft-agents-a365-observability-core/microsoft_agents_a365/observability/core/config.py

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

* Update libraries/microsoft-agents-a365-observability-core/microsoft_agents_a365/observability/core/config.py

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

* Update libraries/microsoft-agents-a365-observability-core/microsoft_agents_a365/observability/core/middleware/baggage_builder.py

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

* Remove redundant null check in conversation ID extraction (#64)

* Initial plan

* Remove redundant condition in _iter_conversation_pairs

Co-authored-by: nikhilNava <211831449+nikhilNava@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: nikhilNava <211831449+nikhilNava@users.noreply.github.com>

* Fix copyright header in utils.py to use standard Microsoft format (#65)

* Initial plan

* Fix copyright header in utils.py to use standard Microsoft format

Co-authored-by: nikhilNava <211831449+nikhilNava@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: nikhilNava <211831449+nikhilNava@users.noreply.github.com>

* Update libraries/microsoft-agents-a365-observability-core/microsoft_agents_a365/observability/core/middleware/baggage_builder.py

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

---------

Co-authored-by: Nikhil Chitlur Navakiran (from Dev Box) <nikhilc@microsoft.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
Co-authored-by: nikhilNava <211831449+nikhilNava@users.noreply.github.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.

2 participants