Skip to content

Fix PAL QueryThreadCycleTime test format macros - #132400

Open
jkotas with Copilot wants to merge 2 commits into
mainfrom
copilot/ci-scan-fix-paltest-compile-error
Open

Fix PAL QueryThreadCycleTime test format macros#132400
jkotas with Copilot wants to merge 2 commits into
mainfrom
copilot/ci-scan-fix-paltest-compile-error

Conversation

Copilot AI commented Aug 17, 2026 •
edited by jkotas
Loading

Copy link
Copy Markdown
Contributor

Fixes #132390

Copilot AI lite review requested due to automatic review settings August 17, 2026 11:10
Copilot AI reviewed Aug 17, 2026
View reviewed changes

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.

Copilot wasn't able to review any files in this pull request.

@azure-pipelines

azure-pipelines Bot commented Aug 17, 2026

Copy link
Copy Markdown
Azure Pipelines:
16 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

Copilot AI linked an issue Aug 17, 2026 that may be closed by this pull request
[ci-scan] Build break: PAL test QueryThreadCycleTime/test1.cpp fails to compile (expected ')') #132390
Open
@dotnet-policy-service

dotnet-policy-service Bot commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

Tagging subscribers to this area: @dotnet/runtime-infrastructure
See info in area-owners.md if you want to be subscribed.

@jkotas
Fix QueryThreadCycleTime PAL test format specifiers
530d019 
Co-authored-by: jkotas <6668460+jkotas@users.noreply.github.com>
Copilot AI review requested due to automatic review settings August 17, 2026 11:37
Copilot AI changed the title [WIP] Fix build break in PAL test QueryThreadCycleTime/test1.cpp Fix PAL QueryThreadCycleTime test format macros Aug 17, 2026
Copilot AI requested a review from jkotas August 17, 2026 11:39
Copilot AI reviewed Aug 17, 2026
View reviewed changes

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

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

@jkotas
jkotas requested a review from jkoritzinsky August 17, 2026 11:42
@jkotas
jkotas marked this pull request as ready for review August 17, 2026 11:43
@azure-pipelines

azure-pipelines Bot commented Aug 17, 2026

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 3 pipeline(s).
13 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

@jkotas jkotas closed this Aug 17, 2026
@jkotas jkotas reopened this Aug 17, 2026
@azure-pipelines

azure-pipelines Bot commented Aug 17, 2026

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 3 pipeline(s).
13 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

This was referenced Aug 17, 2026
Open
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot
Copilot review effort
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments
@jkoritzinsky jkoritzinsky Awaiting requested review from jkoritzinsky
+1 more reviewer
@jkotas jkotas jkotas approved these changes
Reviewers whose approvals may not affect merge requirements

At least 1 approving review is required to merge this pull request.

Assignees

@jkotas jkotas

Copilot code review Copilot

Labels

area-Infrastructure

Projects

Runtime Infra
Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[ci-scan] Build break: PAL test QueryThreadCycleTime/test1.cpp fails to compile (expected ')')

3 participants

@jkotas