Skip to content

Comments

.NET: Python, .NET: [Purview] Mark responses as responses and fix epoch bug for python #4225

Open
Rishabh4275 wants to merge 1 commit intomicrosoft:mainfrom
Rishabh4275:users/richawla/psdkNew
Open

.NET: Python, .NET: [Purview] Mark responses as responses and fix epoch bug for python #4225
Rishabh4275 wants to merge 1 commit intomicrosoft:mainfrom
Rishabh4275:users/richawla/psdkNew

Conversation

@Rishabh4275
Copy link
Contributor

@Rishabh4275 Rishabh4275 commented Feb 24, 2026

…ng overflow

Motivation and Context

Python, .NET: [Purview] Mark responses as responses and fix epoch bug for python long overflow.

Description

Contribution Checklist

  • The code builds clean without any errors or warnings
  • The PR follows the Contribution Guidelines
  • All unit tests pass, and I have added new tests where possible
  • Is this a breaking change? If yes, add "[BREAKING]" prefix to the title of the PR.

Copilot AI review requested due to automatic review settings February 24, 2026 20:27
@github-actions github-actions bot changed the title Python, .NET: [Purview] Mark responses as responses and fix epoch bug for python .NET: Python, .NET: [Purview] Mark responses as responses and fix epoch bug for python Feb 24, 2026
@Rishabh4275 Rishabh4275 force-pushed the users/richawla/psdkNew branch from a71f1ee to 767a4d3 Compare February 24, 2026 20:54
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@markwallace-microsoft
Copy link
Member

markwallace-microsoft commented Feb 25, 2026

Python Test Coverage

Python Test Coverage Report •
FileStmtsMissCoverMissing
packages/purview/agent_framework_purview
   _middleware.py1110100% 
   _processor.py1711094%178, 258–261, 266, 294, 296, 302, 304
TOTAL21808276487% 

Python Unit Test Overview

Tests Skipped Failures Errors Time
4558 244 💤 0 ❌ 0 🔥 1m 15s ⏱️

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants