Skip to content

Fix formatting - #241

Open
Radhika Gupta (rads-1996) wants to merge 1 commit into
microsoft:mainfrom
rads-1996:fix-black
Open

Fix formatting#241
Radhika Gupta (rads-1996) wants to merge 1 commit into
microsoft:mainfrom
rads-1996:fix-black

Conversation

@rads-1996

Copy link
Copy Markdown
Member

No description provided.

@github-actions

github-actions Bot commented Aug 7, 2026

Copy link
Copy Markdown

Performance comparison

Threshold: regressions >15.0% on gating scenarios fail the build. Higher ops/s is better; positive Δ means the PR is slower.

Scenario Gating Baseline (ops/s) Candidate (ops/s) Δ % Status
azure_monitor_log yes 19,341.8 19,139.9 +1.06%
azure_monitor_span yes 156,201.2 154,273.4 +1.25%
otel_log no 22,649.0 22,389.9 +1.16%
otel_span no 38,974.2 38,822.9 +0.39%

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.

Pull request overview

This PR applies formatting-only updates across several test modules and a few runtime modules, aligning code style with the repo’s configured formatter settings (e.g., Black with line-length = 120).

Changes:

  • Rewrapped/condensed multiline expressions (list comprehensions, function calls, and assertions) for consistent formatting.
  • Removed stray blank/whitespace-only lines in a couple of modules.
  • Minor slice-spacing adjustment in token parsing logic.

Reviewed changes

Copilot reviewed 13 out of 14 changed files in this pull request and generated no comments.

Show a summary per file
File Description
tests/a365/test_exporter.py Reformats assertLogs context manager usage.
tests/a365/integration/openai/test_openai_trace_processor.py Condenses model construction and some list comprehensions/conditionals for style consistency.
tests/a365/integration/openai/test_message_format.py Reformats dict appends, list comprehensions, and assertion message formatting.
tests/a365/integration/langchain/test_observability_pipeline.py Reformats assertions and list comprehensions.
tests/a365/integration/langchain/test_message_format.py Condenses list comprehension and assertion formatting.
tests/a365/integration/agentframework/test_observability_pipeline.py Reformats assertions and related formatting blocks.
tests/a365/integration/agentframework/test_message_format.py Condenses helper/list comprehensions and assertion formatting.
tests/a365/integration/agentframework/test_agentframework_trace_processor.py Condenses test function signature and conditional formatting.
src/microsoft/opentelemetry/a365/core/message_utils.py Condenses list comprehension in message serialization.
src/microsoft/opentelemetry/a365/core/exporters/agent365_exporter.py Adjusts slice formatting in Bearer token parsing.
src/microsoft/opentelemetry/_utils.py Condenses a multi-line guard condition into a single line.
src/microsoft/opentelemetry/_sdkstats/_constants.py Removes a stray whitespace-only line.
src/microsoft/opentelemetry/_otlp/handler.py Removes a stray whitespace-only line.
src/microsoft/opentelemetry/_genai/_langchain/_tracer.py Condenses a conditional assignment to a single line.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip-changelog Not required for changes to test files, github actions, etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants